Skip to content

chore: fix README formatting in the OpenFeature server packages - #2046

Closed
tanderson-ld wants to merge 2 commits into
mainfrom
ta/format-openfeature-readmes
Closed

tanderson-ld wants to merge 2 commits into
mainfrom
ta/format-openfeature-readmes

Conversation

@tanderson-ld

@tanderson-ld tanderson-ld commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The openfeature-node-server and openfeature-cloudflare-server READMEs fail format:check, which breaks those packages' path-triggered CI workflows. Because the workflows only run when their paths change, the failure lies dormant on main and surfaces on any PR that touches shared code those packages depend on (first observed on #2045, whose own files are format-clean).

Formatting-only change produced by running the packages' format scripts; two lines total. The repo-wide format:check passes with this applied.


Note

Overview
Applies formatting-only fixes to the READMEs in openfeature-node-server and openfeature-cloudflare-server (two lines total via each package’s format script) so those workspaces pass format:check.

That unblocks path-triggered CI for those packages, which had been failing on main until something touched their paths.

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

@joker23

joker23 commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

@tanderson-ld I think we can close this as the changes were done upstream

@tanderson-ld
tanderson-ld marked this pull request as ready for review September 25, 2026 17:51
@tanderson-ld

Copy link
Copy Markdown
Contributor Author

Superseded: #2044 landed identical formatting for both READMEs on main (byte-identical output from the same formatter), so this branch's diff against main is now empty. Closing; no follow-up needed.

@tanderson-ld
tanderson-ld deleted the ta/format-openfeature-readmes branch September 25, 2026 17:53
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