Skip to content
This repository has been archived by the owner on Jun 6, 2022. It is now read-only.

Support PostCSS 8 #16

Merged
merged 1 commit into from
Jan 31, 2021
Merged

Support PostCSS 8 #16

merged 1 commit into from
Jan 31, 2021

Conversation

Semigradsky
Copy link
Member

No description provided.

@giotramu
Copy link

Any update on this PR?

@michgeek
Copy link

I think this PR should not include the yarn.lock

@Semigradsky
Copy link
Member Author

Why not? We can lock devDependencies
https://classic.yarnpkg.com/blog/2016/11/24/lockfiles-for-all/

@michgeek
Copy link

Because this repo originally didn't have the lock, and I thought there was a reason for it not being checked in. But I agree that dev deps should always be locked, so not sure why it was never checked in initially.

@Airkro
Copy link

Airkro commented Jan 29, 2021

Some people think the lock file should be committed in a team project(or big project), shouldn't commit in a single library repos.

But whatever, don't let the lock file problem block this pr.

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

Successfully merging this pull request may close these issues.

4 participants