Skip to content

jsonc: Recognize renovate.json as JSONC - #54527

Merged
MrSubidubi merged 1 commit into
zed-industries:mainfrom
notpeter:renovate_jsonc
Apr 22, 2026
Merged

jsonc: Recognize renovate.json as JSONC#54527
MrSubidubi merged 1 commit into
zed-industries:mainfrom
notpeter:renovate_jsonc

Conversation

@notpeter

@notpeter notpeter commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

renovate.json supports comments [source]
Support this by default.

Screenshot 2026-04-22 at 2 34 43 PM

Self-Review Checklist:

  • [Yes] I've reviewed my own diff for quality, security, and reliability
  • [N/A] Unsafe blocks (if any) have justifying comments
  • [N/A] The content is consistent with the UI/UX checklist
  • [N/A] Tests cover the new/changed behavior
  • [N/A] Performance impact has been considered and is acceptable

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 22, 2026

@MrSubidubi MrSubidubi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! ❤️

@MrSubidubi
MrSubidubi enabled auto-merge (squash) April 22, 2026 18:34
@MrSubidubi MrSubidubi changed the title jsonc: Recognize renovate.json as JSONC jsonc: Recognize renovate.json as JSONC Apr 22, 2026
@MrSubidubi
MrSubidubi disabled auto-merge April 22, 2026 18:35
@MrSubidubi
MrSubidubi enabled auto-merge (squash) April 22, 2026 18:35
@MrSubidubi
MrSubidubi merged commit 5920187 into zed-industries:main Apr 22, 2026
36 checks passed
MrSubidubi added a commit that referenced this pull request Apr 22, 2026
Reverts #54527

This has unintended side effects, thus revert it to the prior state for
now.

Release Notes:

- N/A
kathbigra pushed a commit to kathbigra/zed that referenced this pull request May 10, 2026
Self-Review Checklist:

- [Yes] I've reviewed my own diff for quality, security, and reliability
- [N/A] Unsafe blocks (if any) have justifying comments
- [N/A] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [N/A] Tests cover the new/changed behavior
- [N/A] Performance impact has been considered and is acceptable

Release Notes:

- N/A
kathbigra pushed a commit to kathbigra/zed that referenced this pull request May 10, 2026
)

Reverts zed-industries#54527

This has unintended side effects, thus revert it to the prior state for
now.

Release Notes:

- N/A
jasonsmithio pushed a commit to paddleboarddev/paddleboard that referenced this pull request May 31, 2026
Reverts zed-industries/zed#54527

This has unintended side effects, thus revert it to the prior state for
now.

Release Notes:

- N/A
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Self-Review Checklist:

- [Yes] I've reviewed my own diff for quality, security, and reliability
- [N/A] Unsafe blocks (if any) have justifying comments
- [N/A] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [N/A] Tests cover the new/changed behavior
- [N/A] Performance impact has been considered and is acceptable

Release Notes:

- N/A
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
)

Reverts zed-industries#54527

This has unintended side effects, thus revert it to the prior state for
now.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Self-Review Checklist:

- [Yes] I've reviewed my own diff for quality, security, and reliability
- [N/A] Unsafe blocks (if any) have justifying comments
- [N/A] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [N/A] Tests cover the new/changed behavior
- [N/A] Performance impact has been considered and is acceptable

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants