Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[2019] Remove legacy commit status override #124

Closed
gr2m opened this issue Oct 13, 2018 · 3 comments · Fixed by #285
Closed

[2019] Remove legacy commit status override #124

gr2m opened this issue Oct 13, 2018 · 3 comments · Fixed by #285
Assignees
Labels

Comments

@gr2m
Copy link
Collaborator

gr2m commented Oct 13, 2018

When a pull request status context has the same name as a check, one does not override the other, instead they are both listed:

image

That means that a previous "pending" commit status can block a pul pull_request indefinitely, as the new version no longer sets commit but uses checks instead.

So for a transition period the app checks if there is a commit status set and overwrites it, unfortunately there is now way to simply delete a status instead. That’s why the app requires the read/write permission for commit status for the same transition period.

I have not yet decided how long this transition period will be, but at least until 2019

follow up for #94

This was referenced Oct 13, 2018
@gr2m gr2m changed the title Remove legacy commit status override [2019] Remove legacy commit status override Oct 26, 2018
@gr2m gr2m self-assigned this May 31, 2019
@gr2m
Copy link
Collaborator Author

gr2m commented Nov 26, 2019

I check every now and then if there are still requests from the legacy account. And yes, even one year after I created the new version which prompted the new permissions, and half a year after I started to nudge users to accept the permissions, there are still hundreds of requests 🤷‍♂ 5 people and organizations accepted the new permissions in the past week alone.

I'll continue to watch the usage numbers, but with the current tendency I think I'll leave the legacy commit status handling for another 6 months at least

@gr2m
Copy link
Collaborator Author

gr2m commented Dec 10, 2020

There were still 323 installations that didn't accept the new permissions after over 2 years, so the probably never will. I uninstalled the WIP app for these installations now, which means I can finally remove the legacy code.

@gr2m gr2m closed this as completed Dec 10, 2020
gr2m added a commit that referenced this issue Dec 10, 2020
gr2m added a commit that referenced this issue Dec 10, 2020
gr2m added a commit that referenced this issue Dec 10, 2020
@github-actions
Copy link

🎉 This issue has been resolved in version 4.10.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant