-
Notifications
You must be signed in to change notification settings - Fork 914
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
chore: remove unused configs #991
chore: remove unused configs #991
Conversation
@@ -1,3 +1,2 @@ | |||
workspaces-experimental "true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Starting Yarn 1.0 Workspaces are enabled by default and you may not need to set the below config. Refer to the updated steps at the following location https://yarnpkg.com/lang/en/docs/workspaces/
2017/08/02
https://classic.yarnpkg.com/blog/2017/08/02/introducing-workspaces/#setting-up-workspaces
off-topic: they released/releasing v2 of yarn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we might have a look. Last I checked the ts interop was not great
"babel": { | ||
"presets": [ | ||
"babel-preset-commitlint" | ||
], | ||
"ignore": [ | ||
"**/*.test.js" | ||
] | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i missed this one last time
os: windows | ||
language: node_js | ||
node_js: | ||
- lts/* | ||
before_script: | ||
- yarn build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
travis is no longer used
"dependencies": { | ||
"@commitlint/config-patternplate": "^8.3.4", | ||
"@commitlint/config-patternplate": "^8.3.4" | ||
}, | ||
"devDependencies": { | ||
"@commitlint/utils": "^8.3.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is no reason to require utils when installing config,
utils is a dev package
Description
Remove no longer used/needed configs and fix
@alias/commitlint-config-patternplate
dependenciesMotivation and Context
Usage examples
How Has This Been Tested?
Types of changes
Checklist: