Skip to content

fix: don't format CHANGELOG.md#35

Merged
nicknisi merged 1 commit into
mainfrom
nicknisi/ignore-changelog-formatting
May 18, 2026
Merged

fix: don't format CHANGELOG.md#35
nicknisi merged 1 commit into
mainfrom
nicknisi/ignore-changelog-formatting

Conversation

@nicknisi
Copy link
Copy Markdown
Member

This pull request makes a minor update to the .oxfmtrc.json configuration file by adding CHANGELOG.md to the list of files ignored by the formatter.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 18, 2026

Greptile Summary

Adds CHANGELOG.md to the ignorePatterns list in .oxfmtrc.json so the oxfmt formatter no longer reformats the changelog file. The array is also expanded to multi-line format for readability.

  • .oxfmtrc.json: CHANGELOG.md appended to ignorePatterns; no other formatter settings changed.

Confidence Score: 5/5

Safe to merge — the change is a one-line addition to a formatter ignore list with no effect on runtime behavior.

Adds a single entry to the formatter's ignore list and expands the array to multi-line format. The change has no impact on application logic, tests, or build output.

No files require special attention.

Important Files Changed

Filename Overview
.oxfmtrc.json Adds CHANGELOG.md to the formatter's ignorePatterns list and reformats the array to multi-line style for readability.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[oxfmt formatter runs] --> B{File matches ignorePatterns?}
    B -- pnpm-lock.yaml / package-lock.json / dist --> C[Skip formatting]
    B -- CHANGELOG.md NEW --> C
    B -- No match --> D[Format file]
Loading

Reviews (1): Last reviewed commit: "fix: don't format CHANGELOG.md" | Re-trigger Greptile

@nicknisi nicknisi merged commit ab8082f into main May 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant