« first day (809 days earlier)      last day (630 days later) » 

11:55 AM
poing
 
@OlegValteriswithUkraine Maybe sloshy should watch this room?
 
@HenryEcker yeah, yeah, I know :)
too lazy right now
 
12:15 PM
PR'd Sloshy finally, although CI failed as it looks like it couldn't log in
 
12:43 PM
@OlegValteriswithUkraine it's an unrelated error, I have been too lazy to figure it out but the basic problem is that PRs don't have access to the production credentials
 
@tripleee yeah, I know - encountered this before as PRs from external forks do not have access to secrets. I guess you can add a workflow_run action that is run with picked up new room ids as artefacts, but that's an amount of work I am not sure is warranted
 
I had some vague ideas for how to test room accesses at startup, then at least I'd get a reasonable warning if the merged PR doesn't work
but it's been a while ... I guess there may also be some hooks to say that pure YAML changes should be safe for getting prod credentials or something like that
 
1:01 PM
@tripleee I suppose an alternative is, indeed, to analyze (using pull_request_target) the content of the PR and, if the change is config-only, proceed, else exit the run.
 
 
7 hours later…
8:14 PM
in SO Close Vote Reviewers, 47 secs ago, by rene
As there are some script/app/library maintainers here and probably a few consumers, we have a post on Stack Apps that you might have input for.
 

« first day (809 days earlier)      last day (630 days later) »