Chad Schulz

Results 9 comments of Chad Schulz

Thank you for the feedback! In general I agree with @AArnott but am interested in hearing more @Evanion. > check if current branch is behind target branch, and then update...

The reference in CI of the "merge" from a pull request is typically something like `refs/pull/15/merge`, which is the "merge of pull request 15 into the main branch". Some CIs...

Nice suggestion, I am 100% open to contributions if you'd like to set that up as a option within the Action!

@aaron-trout and @bartelemi -- no problem with the PRs and issues -- I appreciate the help! I am working on getting your PRs in and then can look at this...

This is an interesting problem... I am doing some research into how this may be handled. Only thing I can think of is if the merge fails for a `Status:...

Is there a way we could provide both experiences? @Evanion, what would you recommend when a non-required reviewer has pending feedback?

@vaga Do you have any thoughts / feedback on this update?

@vaga Do you have any thoughts / feedback on this update?

Thank you for the updates @vaga! I pulled in the latest to my site and the theme is super accessible, [valid](https://validator.w3.org/nu/?doc=https%3A%2F%2Fsqualr.us%2F), and [fast](https://pagespeed.web.dev/report?url=https%3A%2F%2Fsqualr.us%2F&form_factor=mobile)!