Skip to content
This repository has been archived by the owner on Oct 11, 2023. It is now read-only.

Process configuration feature is not used and could be removed #333

Closed
Tracked by #356
joao-paulo-parity opened this issue Oct 17, 2021 · 0 comments · Fixed by #358
Closed
Tracked by #356

Process configuration feature is not used and could be removed #333

joao-paulo-parity opened this issue Oct 17, 2021 · 0 comments · Fixed by #358

Comments

@joao-paulo-parity
Copy link
Contributor

The "process configuration" (Process.json) feature is not used by any of our repositories and could be removed.

At the moment the application ends up doing useless API calls to fetch the contents of a configuration which is not used. The extra criteria based on process configuration overall adds complexity for no value and makes the "approvals failed" message more difficult to parse because it explains an extra criterion that no one uses or cares about.

Even though the feature might sound useful for flexibility's sake, I believe we'd rather have CODEOWNERS rules, which is natively integrated into Github, rather than a Process.json file which only clutters the repositories and this bot's code.

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

Successfully merging a pull request may close this issue.

1 participant