Skip to content

feat: make HTTP ref resolving be able to switched off#839

Open
pyrooka wants to merge 3 commits into
mainfrom
nb/httpref-config
Open

feat: make HTTP ref resolving be able to switched off#839
pyrooka wants to merge 3 commits into
mainfrom
nb/httpref-config

Conversation

@pyrooka

@pyrooka pyrooka commented Jul 24, 2026

Copy link
Copy Markdown
Member

PR summary

This PR adds a new configuration option (through config file or CLI option)
which allows the users to disable the resolving of HTTP references during
the document processing.
Furthermore, a new section has been added to the README which clarifies
what kind of consumption of the project is supported.

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 July 24, 2026 14:37
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka
pyrooka force-pushed the nb/httpref-config branch from ca1a3e1 to e42ca9d Compare July 24, 2026 14:16
@pyrooka
pyrooka requested review from Andris28 and diatrcz July 24, 2026 14:16

@Andris28 Andris28 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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