anaynayak / buildnotify

A system tray based build status notification app for cctray.xml feeds.
https://anaynayak.github.io/buildnotify/
Other
29 stars 7 forks source link

do not auto-subscribe to new branches unless all were already selected #53

Closed MartinNowak closed 4 years ago

MartinNowak commented 6 years ago

At least semaphore CI only has a single cc.xml endpoint that lists build statuses for all branches (https://github.com/renderedtext/semaphore-docs-new/blob/a9a2f51eca9f624a8a8d00ba320b7a87c467d544/source/docs/cctry.md). If you're only interested in master and stable, but not in pull requests, then it's possible to only select those. grafik Unfortunately at the time new pull requests are opened, they are listed by the endpoint and buildnotify automatically selects/subscribes to those.

I'd suggest that the All state should be saved separately (if not already done) and be used to decide whether or not to auto-subscribe to new branches.

anaynayak commented 6 years ago

Thanks @MartinNowak for reporting this. Makes sense. Will try to work on it some time this week.

Deuchnord commented 4 years ago

@anaynayak is it ok if I take this? :slightly_smiling_face:

anaynayak commented 4 years ago

Yes please. Thanks @Deuchnord

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.