Skip to content
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

Make prettier ignore codemods fixtures #387

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

martimalek
Copy link
Contributor

What:

Makes prettier ignore codemods fixtures


Why:

For each codemod test there are 2 fixture files, an *.input.tsx and an *.output.tsx, the test executes the codemod against the input file and compares the result with the output file, it only succeeds when they match. Since codemods are not expected to format the result code for you (since every project has different code styles) in case we format the input or the output files they could stop matching and provoke a failing test.


How:

  • Add the path to the codemods fixtures in .prettierignore


Checklist:

  • Ready to be merged

@martimalek martimalek self-assigned this Oct 4, 2023
@martimalek martimalek mentioned this pull request Oct 4, 2023
4 tasks
@nlopin nlopin merged commit 9fcdbf6 into next Oct 4, 2023
10 checks passed
@nlopin nlopin deleted the prettier-ignore-codemods-fixtures branch October 4, 2023 16:28
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.

2 participants