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 order for plugin set with config and cli #319

Merged
merged 1 commit into from
Aug 19, 2020

Conversation

giuseppeg
Copy link
Contributor

Fixes #318

Plugins defined via CLI options take precedence over the ones from config file.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bb3973a on giuseppeg:fix/plugin-order into 4616152 on posthtml:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bb3973a on giuseppeg:fix/plugin-order into 4616152 on posthtml:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling bb3973a on giuseppeg:fix/plugin-order into 4616152 on posthtml:master.

@@ -23,7 +23,7 @@
"build": "rimraf lib && babel src -d lib",
"coverage": "nyc report --reporter=text-lcov | coveralls",
"pretest": "npm run build",
"test": "nyc ava test/test-cli.js --timeout=1m --verbose"
"test": "nyc ava test/test-*.js --timeout=1m --verbose"
Copy link
Member

Choose a reason for hiding this comment

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

yeah, I once again forgot to return 👍

@Scrum Scrum added this to the 0.7.4 milestone Aug 19, 2020
@Scrum Scrum merged commit af8afb7 into posthtml:master Aug 19, 2020
@Scrum
Copy link
Member

Scrum commented Aug 19, 2020

@giuseppeg publish v0.7.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin order when merging CLI opts and config file
3 participants