Skip to content

chore: format all files with prettier#1902

Closed
Fredkiss3 wants to merge 12 commits intofosrl:devfrom
Fredkiss3:chore/fix-formatting
Closed

chore: format all files with prettier#1902
Fredkiss3 wants to merge 12 commits intofosrl:devfrom
Fredkiss3:chore/fix-formatting

Conversation

@Fredkiss3
Copy link
Copy Markdown
Contributor

@Fredkiss3 Fredkiss3 commented Nov 21, 2025

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

Main changes:

Other changes:

  • Added a job that checks that all files in are formatted correctly with prettier
  • Added a job to run formatting accross the whole repo: npm run format
  • Added .vscode/settings.json with some default options: setting prettier as the default formatter, and formatting on save
  • Added .vscode/extensions.json with recommendations for new contributors recommending them to install prettier extension if not installed

@oschwartz10612
Copy link
Copy Markdown
Member

@Fredkiss3 because of the merge conflicts as we get ready for this next release could you redo this PR or adjust it so that we are not formatting all of the files but you just include your changes for the npm run command and other vscode defaults? Then we will format once things are all merged up outside of a PR.

@Fredkiss3
Copy link
Copy Markdown
Contributor Author

@oschwartz10612 noted

@Fredkiss3 Fredkiss3 closed this Dec 8, 2025
@Fredkiss3
Copy link
Copy Markdown
Contributor Author

@oschwartz10612 check #2016 for the VSCode and changes and npm run changes

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