Skip to content

chore(js): update fix:fmt config #2514

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 26, 2025
Merged

chore(js): update fix:fmt config #2514

merged 1 commit into from
Mar 26, 2025

Conversation

ali-behjati
Copy link
Collaborator

@ali-behjati ali-behjati commented Mar 21, 2025

Summary

  • update turbo fix config in pre-commit: the affected config works in identifying changes between commits and in pre-commit we are dealing with a dirty (unstaged, staged) changes.
  • add submodules path to prettierignore. one package in a submodules was modified with the top level prettier and that strangely resulted in prettier giving a strange error.

@ali-behjati ali-behjati requested a review from a team as a code owner March 21, 2025 16:30
Copy link

vercel bot commented Mar 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Skipped Deployments
Name Status Preview Comments Updated (UTC)
api-reference ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm
component-library ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm
entropy-debugger ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm
insights ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm
proposals ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm
staking ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2025 1:55pm

- update turbo config in precommit
- add submodules path to prettierignore
@ali-behjati ali-behjati merged commit 74d1fc8 into main Mar 26, 2025
13 checks passed
@ali-behjati ali-behjati deleted the js-improve-fix-fmt branch March 26, 2025 15:12
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