Skip to content

docs: fix custom ruleset filenames in README#789

Merged
pyrooka merged 2 commits intomainfrom
nb/fix-readme
Mar 5, 2026
Merged

docs: fix custom ruleset filenames in README#789
pyrooka merged 2 commits intomainfrom
nb/fix-readme

Conversation

@pyrooka
Copy link
Member

@pyrooka pyrooka commented Mar 4, 2026

PR summary

PR Checklist

General checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Dependencies have been updated as needed
  • .secrets.baseline has been updated as needed
  • npm run update-utilities has been run if any files in packages/utilities/src have been updated

Checklist for adding a new validation rule:

  • Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
  • If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
  • Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
  • Added tests for new rule (packages/ruleset/test/*.test.js)
  • Added docs for new rule (docs/ibm-cloud-rules.md)
  • Added scoring rubric entry for new rule (packages/validator/src/scoring-tool/rubric.js)

pyrooka added 2 commits March 4, 2026 19:31
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka pyrooka requested review from Andris28 and diatrcz March 4, 2026 18:52
Copy link
Contributor

@diatrcz diatrcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@pyrooka pyrooka merged commit 5533069 into main Mar 5, 2026
7 checks passed
@pyrooka pyrooka deleted the nb/fix-readme branch March 5, 2026 09:51
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