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

move browserslist to package.json, use more modern default browsers #43

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

secured
Copy link

@secured secured commented Sep 30, 2020

  Replace Autoprefixer browsers option to Browserslist config.
  Use browserslist key in package.json or .browserslistrc file.

  Using browsers option can cause errors. Browserslist config
  can be used for Babel, Autoprefixer, postcss-normalize and other tools.

@machwatt
Copy link

machwatt commented Nov 6, 2020

Would love to have that implemented as the autoprefixer build step throws that warning. I personally prefer the .browserslistrc file to have that config isolated from the more general package.json file, but that is just my opinion, either way is better than the warning in the grunt task.

@secured
Copy link
Author

secured commented Jan 8, 2021

I personally prefer the .browserslistrc file to have that config isolated from the more general package.json file, but that is just my opinion, either way is better than the warning in the grunt task.

Good idea.

I replaced node-sass with sass because the package is deprecated. I also got problems when compiling with modern sass
@benjaminkott

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.

2 participants