Porting probot-config into Probot proper
See original GitHub issueWhy do we need org wide configuration?
For some history on org wide configuration and why it’s needed check out this dinosaur: https://github.com/probot/probot/issues/258
Current state of things
Right now org wide configuration require installing a separate probot extension, probot-config.
We do, however, have a built-in internal API for getting a config file for the repo an App is acting on. That method is context.config()
and it takes a file name (located in the .github/
folder) and loads the contents of the yml file into an object.
Any time an app wants to add support for org-wide management, they have to require probot-config
and using that to load their configuration files.
The future
It would be pretty amazing if org support was just built into probot, so that no apps need to update their code to support it. (aside from bumping their probot version?)
What this would look like: context.config()
method would use probot-config
under the hood.
Would love any 💭 or 👍 👎 🎉 ❤️ 😆
cc/ @probot/maintainers cc/ @jan-auer (original probot-config creator)
Issue Analytics
- State:
- Created 5 years ago
- Reactions:5
- Comments:12 (10 by maintainers)
Top GitHub Comments
The Welcome app is still running on Probot 7, the source code is here: https://github.com/behaviorbot/welcome/
It’d be great to update it and the other apps it depends on to the latest Probot. Once we have that working, we can setup continuous deployment. Bex added me as admin to the @behaviorbot org so I merge PRs
I do not know that, I’m actually not sure how these are deployed. But find out. Maybe ideally they would be deployed with each new GitHub version? We could pin the probot version in the dependencies and use Greenkeeper to update to the latest probot with each release. As long as automated deployment is setup this should just work ™️