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

fix(js): Adds mjs files to prettierrcNameOptions #21796

Merged
merged 2 commits into from
May 10, 2024

Conversation

ppfenning92
Copy link
Contributor

@ppfenning92 ppfenning92 commented Feb 13, 2024

Current Behavior

when running a generator, it does not check for the existence of a prettier.config.mjs or .prettierrc.mjs file

Expected Behavior

When using a generator, it check for the existence of all config file names specified in the docs.

Related Issue(s)

Fixes #21795

adds .mjs file names to checks

@ppfenning92 ppfenning92 requested a review from a team as a code owner February 13, 2024 08:26
Copy link

vercel bot commented Feb 13, 2024

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

1 Ignored Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Apr 7, 2024 11:15am

Copy link

nx-cloud bot commented Feb 13, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 16ba92c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx run-many -t e2e,e2e-macos -p e2e-gradle
nx run-many -t e2e,e2e-macos -p e2e-workspace-create
nx run-many -t e2e,e2e-macos -p e2e-nx-run
nx run-many -t e2e,e2e-macos -p e2e-release
nx run-many -t e2e,e2e-macos -p e2e-nx-misc
nx run-many -t e2e,e2e-macos -p e2e-react-native
✅ Successfully ran 36 targets

Sent with 💌 from NxCloud.

@ppfenning92 ppfenning92 changed the title Adds mjs files to prettierrcNameOptions fix(nx-dev): Adds mjs files to prettierrcNameOptions Mar 15, 2024
@ppfenning92 ppfenning92 changed the title fix(nx-dev): Adds mjs files to prettierrcNameOptions fix(core): Adds mjs files to prettierrcNameOptions Mar 17, 2024
@ppfenning92 ppfenning92 changed the title fix(core): Adds mjs files to prettierrcNameOptions fix(js): Adds mjs files to prettierrcNameOptions Mar 17, 2024
Copy link
Contributor

@ndcunningham ndcunningham left a comment

Choose a reason for hiding this comment

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

LGTM.
You may have to rebase.

@ndcunningham ndcunningham merged commit 88ac601 into nrwl:master May 10, 2024
5 checks passed
FrozenPandaz pushed a commit that referenced this pull request May 13, 2024
<!-- Please make sure you have read the submission guidelines before
posting an PR -->
<!--
https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr
-->

<!-- Please make sure that your commit message follows our format -->
<!-- Example: `fix(nx): must begin with lowercase` -->

## Current Behavior
<!-- This is the behavior we have today -->
when running a generator, it does not check for the existence of a
prettier.config.mjs or .prettierrc.mjs file

## Expected Behavior
<!-- This is the behavior we should expect with the changes in this PR
-->
When using a generator, it check for the existence of all config file
names specified in the docs.

## Related Issue(s)
<!-- Please link the issue being fixed so it gets closed when this is
merged. -->

Fixes #21795

adds .mjs file names to checks

(cherry picked from commit 88ac601)
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Init generator ignores prettier.config.mjs files, when checking for prettier
4 participants