Skip to content

docs: add docs about duplicating rules and bundle components#2812

Merged
AlbinaBlazhko17 merged 4 commits into
mainfrom
docs/add-docs-about-new-rules-and-bundle
May 14, 2026
Merged

docs: add docs about duplicating rules and bundle components#2812
AlbinaBlazhko17 merged 4 commits into
mainfrom
docs/add-docs-about-new-rules-and-bundle

Conversation

@AlbinaBlazhko17
Copy link
Copy Markdown
Contributor

@AlbinaBlazhko17 AlbinaBlazhko17 commented May 13, 2026

What/Why/How?

  • Added to CONTRIBUTING.md file sentence about naming convention, when rule already exists in other specs.
  • Added to general docs sentence to bundle about manually adding components.

Reference

Testing

Screenshots (optional)

Check yourself

  • This PR follows the contributing guide
  • All new/updated code is covered by tests
  • Core code changed? - Tested with other Redocly products (internal contributions only)
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update has been considered

Security

  • The security impact of the change has been considered
  • Code follows company security practices and guidelines

Note

Low Risk
Low risk documentation-only change with no runtime code or behavior modifications.

Overview
Clarifies contributor guidance for built-in rule additions by recommending reusing existing rule names across specification flavors to keep concepts discoverable.

Updates the bundle command documentation to explicitly state that referenced components are resolved and included automatically, without needing manual definitions.

Reviewed by Cursor Bugbot for commit 59cacd5. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 13, 2026

⚠️ No Changeset found

Latest commit: df6d591

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

CLI Version Mean Time ± Std Dev (s) Relative Performance (Lower is Faster)
cli-latest 1.598s ± 0.039s ▓ 1.02x
cli-next 1.569s ± 0.023s ▓ 1.00x (Fastest)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 80.68% (🎯 80%) 7194 / 8916
🔵 Statements 80.07% (🎯 80%) 7473 / 9333
🔵 Functions 83.68% (🎯 83%) 1436 / 1716
🔵 Branches 72.36% (🎯 72%) 4887 / 6753
File CoverageNo changed files found.
Generated in workflow #9816 for commit df6d591 by the Vitest Coverage Report Action

@AlbinaBlazhko17 AlbinaBlazhko17 marked this pull request as ready for review May 14, 2026 07:52
@AlbinaBlazhko17 AlbinaBlazhko17 requested review from a team as code owners May 14, 2026 07:52
Comment thread docs/@v2/commands/bundle.md Outdated

The `bundle` command differs from the [`join`](./join.md) command.
The `bundle` command takes a root OpenAPI file as input and follows the `$ref` mentions to include all the referenced components into a single output file.
The `bundle` command takes a root OpenAPI file as input and follows the `$ref` mentions to include all the referenced components into a single output file. All components are automatically resolved and included without requiring explicit definitions.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please use semantic breaks, at leas for the newly added text.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks, did this.

@AlbinaBlazhko17 AlbinaBlazhko17 merged commit 68e6c58 into main May 14, 2026
44 checks passed
@AlbinaBlazhko17 AlbinaBlazhko17 deleted the docs/add-docs-about-new-rules-and-bundle branch May 14, 2026 12:49
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.

3 participants