Skip to content

Fix order for plugin set with config and cli - #319

Merged
Scrum merged 1 commit into
posthtml:masterfrom
giuseppeg:fix/plugin-order
Aug 19, 2020
Merged

Scrum merged 1 commit into
posthtml:masterfrom
giuseppeg:fix/plugin-order

Conversation

@giuseppeg

Copy link
Copy Markdown
Contributor

Fixes #318

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

@coveralls

Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown

Coverage Status

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

Comment thread package.json
"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
Copy Markdown
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

Scrum commented Aug 19, 2020

Copy link
Copy Markdown
Member

@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