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: update dependency postcss-normalize-charset to v5 (master) #1721

Merged
merged 2 commits into from Aug 30, 2021

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 10, 2021

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
postcss-normalize-charset ^4.0.1 -> ^5.0.0 age adoption passing confidence

Release Notes

cssnano/cssnano

v5.0.1

Compare Source

v5.0.0

Compare Source

Major changes

  • requires Node >= 10.13
  • PostCSS 8 API, so cssnano 5 does not emit warnings when running under PostCSS 8
  • updated to SVGO 2, fixing many SVG minification bugs
  • updated css-value-parser and css-selector-parser, fixing many bugs
Upgrade notes

If you use the cssnano JavaScript API, you need to change your code:

  • Replace cssnano.process() with cssnano().process() (notice the () after cssnano)

  • pass cssnano options to cssnano() instead of process()

cssnano(cssnanoOptions).process(postcssOptions)

Bug fixes

  • fix improperly discarding @font-face declarations #​726
  • partially fix some isues where cssnano did not combine rules when used together with postcss-nested #​1004
  • fix translate3d() minification #​920
  • fix minification of values starting with e #​589, #​984
  • fix minification of percentage vaalues #​962, #​957
  • fix minification of aspect-ratio #​963
  • fix merging of @supports rules #​974
  • fix sorting of longhand and shorthand properties #​535
  • remove vulnerable dependency and always warn with bad SVG input #​1034

Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot added the 📦 master Apply this label to a pull request, if it has to be cherry-picked to the maste-branch. label Apr 10, 2021
@herschel666
Copy link
Contributor

wait for webpack@5.

@herschel666 herschel666 marked this pull request as draft April 13, 2021 07:15
@renovate renovate bot force-pushed the renovate/master-postcss-normalize-charset-5.x branch from 6acfcfc to fbc4a8c Compare June 28, 2021 07:24
@herschel666 herschel666 marked this pull request as ready for review June 29, 2021 06:22
@renovate renovate bot force-pushed the renovate/master-postcss-normalize-charset-5.x branch from fbc4a8c to 11e3b18 Compare August 25, 2021 07:12
@robertkowalski
Copy link
Contributor

hmmm... version 5 of this node module was fixed for postcss 8 which now prints a warning on invalid imports

postcss/postcss-import#447 explicitly adds a warning for the testcase we have in 3f420ca and does not add a "sloppy" option to skip those warnings

@robertkowalski
Copy link
Contributor

@ZauberNerd we can add a silent option to postcss import if suppressing the warning is really required. lets check on monday, not clear 100% why we don't warn for invalid css

previous versions of normalize-css did not log an error message, with this version
error warnings are prited each time for invalid css.

after double checking the bug reports in slack and also the initial PR, we came to
the conclusion that we should default to print the error warning - especially as
there is no "silent" option in normalize-css available.

Co-authored-by: Björn Brauer <bjoern.brauer@new-work.se>
@robertkowalski
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 30, 2021

Build succeeded:

@bors bors bot merged commit c64bcc7 into master Aug 30, 2021
@bors bors bot deleted the renovate/master-postcss-normalize-charset-5.x branch August 30, 2021 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 master Apply this label to a pull request, if it has to be cherry-picked to the maste-branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants