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

update default repolinter ruleset #5

Merged
merged 1 commit into from
Sep 22, 2021

Conversation

willnorris
Copy link
Member

This change updates our default ruleset in the following ways:

  • move to new v2 syntax
  • switch from json to yaml, mainly for comments support
  • remove code of conduct rule, since it's no longer needed in each repo
  • remove a few other rules, limiting it to the most basic that we are certain all repos should pass (have a README and LICENSE)
  • move into repolinter-rulesets directory, which provides a place to put other extended rulesets that we may want to have apply to certain types of repos, such as those with source code versus simply documentation.

An example of how this would be expected to be used can be seen in some of our Twitter repos:

Relates to todogroup/working-hours#8

This change updates our default ruleset in the following ways:

 - move to new v2 syntax
 - switch from json to yaml, main for commenting support
 - remove code of conduct rule, since it's no longer needed in each repo
 - remove a few other rules, limiting it to the most basic that we are
   certain all repos should pass (have a README and LICENSE)
 - move into repolinter-rulesets directory, which provides a place to
   put other extended rulesets that we may want to have apply to certain
   types of repos, such as those with source code versus simply
   documentation.

We can of course continue to expand this ruleset, but this starts with
something very basic.
@DuaneOBrien DuaneOBrien merged commit 332ae15 into todogroup:master Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants