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

Upgrade styled componenets to v6 #1039

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

jtmiclat
Copy link
Contributor

@jtmiclat jtmiclat commented Dec 6, 2023

Closes #996

This PR updated styled components to v6.

In v5, styled components had no type checking by default while in v6 it is included by default so I had to add typing to the styled components props.

@nyurik
Copy link
Member

nyurik commented Dec 6, 2023

i think you are missing the changes to package.json - did you modify that file too?

@jtmiclat jtmiclat force-pushed the feature/upgrade-styled-components branch from 0e7bba9 to daef924 Compare December 6, 2023 06:10
@jtmiclat
Copy link
Contributor Author

jtmiclat commented Dec 6, 2023

@nyurik Woops forgot to commit package.json . Should be fixed now

@nyurik
Copy link
Member

nyurik commented Dec 6, 2023

Thanks @jtmiclat ! I added a CI step to build the code, so seems to be passing. Would be great to have some sort of a integration test, but not worth it for a demo site.

@nyurik nyurik enabled auto-merge (squash) December 6, 2023 06:42
@nyurik nyurik disabled auto-merge December 6, 2023 06:43
@nyurik nyurik merged commit 958d5a1 into maplibre:main Dec 6, 2023
3 checks passed
@nyurik
Copy link
Member

nyurik commented Dec 6, 2023

@jtmiclat if you have any energy, feel free to bump all other demo dependencies -- it is not as often maintained as one could hope :)

@jtmiclat jtmiclat deleted the feature/upgrade-styled-components branch December 6, 2023 09:54
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.

Upgrade demo site's styled-components NPM package to v6
2 participants