Skip to content

Add oxfmt to the repo#100

Merged
Andarist merged 3 commits intochangesets:masterfrom
marcalexiei:format
Apr 1, 2026
Merged

Add oxfmt to the repo#100
Andarist merged 3 commits intochangesets:masterfrom
marcalexiei:format

Conversation

@marcalexiei
Copy link
Copy Markdown
Contributor

@marcalexiei marcalexiei commented Mar 30, 2026


  • Add oxfmt as formatter
  • Apply format to source code
  • Add format:check script run on ci.yml

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

@marcalexiei is attempting to deploy a commit to the Changesets Team on Vercel.

A member of the Team first needs to authorize it.

@Andarist
Copy link
Copy Markdown
Member

@marcalexiei could you rebase this now? Let's see if the CI checks actually work now :p

@marcalexiei marcalexiei marked this pull request as ready for review March 31, 2026 22:53
@marcalexiei marcalexiei requested a review from emmatown as a code owner March 31, 2026 22:53
Copy link
Copy Markdown
Contributor Author

@marcalexiei marcalexiei left a comment

Choose a reason for hiding this comment

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

could you rebase this now?

Done, this PR is now in sync with master


Let's see if the CI checks actually work now :p

Everything works as expected!
I saw few warnings in the PR CI run and I've created separate threads to discuss them.

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.

> warning " > vitest@4.1.2" has incorrect peer dependency "vite@^6.0.0 || ^7.0.0 || ^8.0.0".

Likely caused by

bot/package.json

Lines 39 to 41 in be84759

"resolutions": {
"vite": "npm:vite@8.0.0-beta.18"
},

Can we install vite latest version (8.0.3) to remove it or there is a particular reason to use this beta?

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.

Copy link
Copy Markdown

@beeequeue beeequeue Apr 1, 2026

Choose a reason for hiding this comment

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

yes, with vitest 4.1 we can remove the override

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.

Done via 1ebb853

- name: Setup dependencies
uses: ./.github/actions/ci-setup

- name: Run format:check
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.

let's match the style in the main Changesets repo where "Lint" is a separate job and it's required by CI OK

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.

Done via 5bea024

@marcalexiei marcalexiei requested a review from Andarist April 1, 2026 09:01
@Andarist Andarist changed the title chore: add formatter Add oxfmt to the repo Apr 1, 2026
@Andarist Andarist added this pull request to the merge queue Apr 1, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
changesets-bot Ready Ready Preview, Comment Apr 1, 2026 9:13am

Request Review

Merged via the queue into changesets:master with commit 6b5ebbc Apr 1, 2026
5 checks passed
@marcalexiei marcalexiei deleted the format branch April 1, 2026 11:16
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