-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Make AutoMerge guidelines checks part of the public API #438
Comments
I believe we've historically only added new checks as either turned-off by default, or as a breaking release. So we've been treating it as part of the public API, as it is the functionality of what |
Right now My main concern was running this on a new hosting provider that doesn't exist in the list (GitLab). When I wrote this, the While the Factoring out the GitHub API object using something like the GitForge.jl abstraction would be the best case for us, but stable lower-level access allows us to start using some of the guidelines that don't depend on API access. |
Ahh ok, thanks for explaining, I misunderstood. Makes sense! |
Right now, the public API only includes
run
, which couples the checks to the interactions with the git hosting service. For private registries, it would be nice to be able to run each guideline and get results, and depend on the ability to do that as well as the guidelines themselves as part of the public API, in order to avoid pinning RegistryCI.jl to an old version.The text was updated successfully, but these errors were encountered: