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.

Sign up for GitHub

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

Jump to bottom

feat!: migrate to postcss 8 #22

Closed
wants to merge 2 commits into from
Closed

feat!: migrate to postcss 8 #22

wants to merge 2 commits into from

Conversation

AviVahl
Copy link

@AviVahl AviVahl commented Oct 3, 2020

  • adjust implementation to postcss' new tranformer api.
  • drop Node 6/8 support. adjusted travis/babel configs. added "engines" field to package.json.
  • migrate to babel@7. use esm -> cjs without additional transformations. commonjs consumers will use .default.
  • upgrade all dependencies to latest stable.
  • move postcss to "peerDependencies".
  • add package-lock.json to repository.

AviVahl added 2 commits October 4, 2020 01:10
@AviVahl
feat!: migrate to postcss 8
9b2f878 
- adjust implementation to postcss' new tranformer api.
- drop Node 6/8 support. adjusted travis/babel configs. added "engines" field to package.json.
- migrate to babel@7.
- upgrade all dependencies to latest stable.
- move postcss to "peerDependencies".
- add package-lock.json to repository.
@AviVahl
fix: eslintignore file identified as symlink
b04da00
@Semigradsky Semigradsky mentioned this pull request Oct 4, 2020
Closed
@onigoetz onigoetz mentioned this pull request Oct 4, 2020
Merged
58 tasks
@AviVahl
Copy link
Author

AviVahl commented Mar 20, 2021

Closing due to no response.

@AviVahl AviVahl closed this Mar 20, 2021
jonathantneal
jonathantneal reviewed Sep 16, 2021
View reviewed changes
Copy link
Member

@jonathantneal jonathantneal left a comment

Choose a reason for hiding this comment

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

This is good.

@AviVahl
Copy link
Author

AviVahl commented Sep 16, 2021

Thanks.

Feel free to take any of these changes and apply them to current master. Further updates may be required, as this was opened over a year ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonathantneal jonathantneal jonathantneal left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AviVahl @jonathantneal