EIT-ALIVE / eitprocessing

Software for electrical impedance tomography data loading, visualization and processing
https://eit-alive.github.io/eitprocessing/
Apache License 2.0
5 stars 1 forks source link

Mark GitHub release branches as merged instead of closed #335

Open DaniBodor opened 3 weeks ago

DaniBodor commented 3 weeks ago

the github release action currently marks PRs as "closed" as opposed to "merged", which is confusing because that makes it look like changes were not accepted.

psomhorst commented 2 weeks ago

I just ran into this as well. I think this is a result of how the github_release action works. Currently, it manually merges, and then later closes the PR.

Could we rewrite this to:

Tidy could then probably be simplified to merging main into develop using https://github.com/marketplace/actions/merge-branch.

DaniBodor commented 2 weeks ago

I already have a PR open for this. The reason is that I explicitly added a step to close the PR, which was unnecessary. I changed it in that PR and I think that solves the issue. The action you found might simplify the entire process (looking at it for a few seconds, it seems like either way can be done with that action). At this point, I will leave the PR as is, but if you prefer doing it with this action, then feel free to redo my PR or add on to it.

psomhorst commented 2 weeks ago

I hadn't noticed you were already working on this, so I had a crack at it. See #338. Note: this is untested.

Please feel free to ignore.