Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

fix(formatting): fix remove code formatting - fixes: #40 #41

Closed
wants to merge 3 commits into from

Conversation

emahuni
Copy link

@emahuni emahuni commented Sep 2, 2022

See issue #40 for more details...

@@ -1,8 +1,9 @@
module.exports = {
Copy link
Member

Choose a reason for hiding this comment

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

Can we keep this file please as it's for this repo, not projects being migrated.

@@ -32,8 +32,7 @@
"inquirer-fuzzy-path": "^2.3.0",
"jscodeshift": "^0.13.0",
"lodash": "^4.17.21",
"pluralize": "^8.0.0",
"prettier": "^2.4.1"
Copy link
Member

Choose a reason for hiding this comment

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

Same here can we keep prettier for this repo

@remidej remidej requested a review from markkaylor September 5, 2022 10:01
@remidej remidej added the pr: fix This PR is fixing a bug label Sep 5, 2022
Comment on lines +3 to +8
semi: false,
singleQuote: false,
quoteProps: 'preserve',
tabWidth: 2,
trailingComma: 'es5',
printWidth: 100,
printWidth: 120,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please don't change these values they are set to match the config we use in the Strapi core repo

@derrickmehaffy
Copy link
Member

Since there has been no update to this PR I'm going to mark it as closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: fix This PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants