Skip to content

Commit

Permalink
fix: leave prettier as is
Browse files Browse the repository at this point in the history
  • Loading branch information
mshanemc committed Oct 30, 2023
1 parent 77d3daf commit b1bf1f5
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions utils/write-dependencies.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ module.exports = (projectPath) => {
const eslintConfigSfTsPjson = require('eslint-config-salesforce-typescript/package.json');

/** devDeps that *should* be in every repo dev-scripts touches. */
const requiredDeps = [].concat(scripts.format ? ['prettier', '@salesforce/prettier-config'] : []);
const requiredDeps = [];

/** These packages are provided to orgs via devScripts. They should not be in the pjson of the consumer
* If you don't like the devScripts version, you can specify your own in sf-dev-rc.json in the dependencies section.
Expand All @@ -95,6 +95,7 @@ module.exports = (projectPath) => {
'typedoc',
'husky',
'pretty-quick',
'prettier',
'nyc',
'ts-node',
'mocha',
Expand All @@ -105,11 +106,12 @@ module.exports = (projectPath) => {
'@types/mocha',
'@types/node',
'@types/sinon',
'@salesforce/dev-config',
'@salesforce/prettier-config',
// this repo manages all things eslint. Its dependencies are in dev-scripts and therefore should be omitted
'eslint-config-salesforce-typescript',
...Object.keys(eslintConfigSfTsPjson.devDependencies),

// leave these along if the project has them
...Object.keys(eslintConfigSfTsPjson.dependencies),
// leave these alone if the project has them
].filter((dep) => !new Set(config.devDepOverrides).has(dep));
/**
* We don't want these in any repo. This is a good way to clean up things en masse
Expand Down

0 comments on commit b1bf1f5

Please sign in to comment.