Skip to content

Conversation

@jeremyfiel
Copy link
Contributor

updates #1830

What/Why/How?

Add Overlay linting guides

Reference

Testing

Needs a screenshot from github actions. i don't use them so don't know how to create one. Can you please add one?

Screenshots (optional)

Check yourself

  • Code changed? - Tested with Redoc/Realm/Reunite (internal)
  • All new/updated code is covered by tests
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update considered

Security

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

@jeremyfiel jeremyfiel requested a review from a team as a code owner December 17, 2025 23:23
@changeset-bot
Copy link

changeset-bot bot commented Dec 17, 2025

🦋 Changeset detected

Latest commit: 6ced875

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@redocly/cli Patch
@redocly/openapi-core Patch
@redocly/respect-core Patch

Not sure what this means? Click here to learn what changesets are.

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

@jeremyfiel jeremyfiel force-pushed the feat/overlay-linting-guides branch from a9fc155 to 6ced875 Compare December 17, 2025 23:29
@tatomyr tatomyr changed the title feat(docs): add Overlay linting guides docs: add Overlay linting guides Dec 19, 2025

The default setting for this rule (in the `recommended` and `minimal` configuration) is `error`.

This is an essential rule. Do not turned it off, except in rare and special cases.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This is an essential rule. Do not turned it off, except in rare and special cases.
This is an essential rule. Do not turn it off, except in rare and special cases.

@adamaltman
Copy link
Member

adamaltman commented Dec 19, 2025

Is there overlay bundling here (a decorator) too?

@jeremyfiel
Copy link
Contributor Author

Don't believe bundling is implemented.

I copied this content from the arazzo version so I'll make those updates to the text too

"@redocly/openapi-core": patch
---

Add Overlay linting guides.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Add Overlay linting guides.
Added a linting guides for OpenAPI Overlay.


The default setting for this rule (in the `recommended` and `minimal` configuration) is `error`.

This is an essential rule. Do not turned it off, except in rare and special cases.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This is an essential rule. Do not turned it off, except in rare and special cases.
This is an essential rule. Do not turn it off, except in rare and special cases.

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