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

chore: Remove CI for unsupported Node versions #2970

Closed
wants to merge 7 commits into from

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Sep 26, 2020

Was trying a larger branch/PR for #2965, but ended up having too much diff when I hit an issues. Will just submit these in smaller chunks for now and figure out what to cherry-pick from v5 separately

@nschonni
Copy link
Contributor Author

@xzyfer any clue why the sass.type tests would be failing on Node 14 only? I think this was the issue I ran into before on my v5-lite branch

@nschonni
Copy link
Contributor Author

Oh, right, I think I hit this and tried to fix this in the other branch. the assert behaviour was depricated in previous versions, and must be updated in 14

@nschonni
Copy link
Contributor Author

OK, not the assert behaviour changes. Looks like the toString is actually returning something different in the current v14 than previous node versions

@nschonni
Copy link
Contributor Author

Closing since landed version pinning to 14.5 for now. New PR should be sent to fix test and unpin

@nschonni nschonni closed this Sep 27, 2020
@nschonni nschonni deleted the unsupported-CI branch September 27, 2020 04:03
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.

None yet

1 participant