Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Bump styled components #3180

Closed
wants to merge 10 commits into from
Closed

Bump styled components #3180

wants to merge 10 commits into from

Conversation

mykeels
Copy link
Contributor

@mykeels mykeels commented Feb 25, 2020

Resolves #3054

Overall change: A very high-level summary of easily-reproducible changes that can be understood by non-devs.

Code changes:

  • Update styled-components version to ^5.0.0.
  • Update jest-styled-components version to ^7.0.0.
  • Updated jest snapshots (lots of changes).

  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

Ikechi Michael added 3 commits February 25, 2020 12:03
- bump jest-styled-components from ^6.3.3 to ^7.0.0
- bump styled-components from ^4.4.1 to ^5.0.0
@mykeels mykeels added ws-home Tasks for the WS Home Team ws-articles Tasks for the WS Articles Team ws-media The World Service media stream Heracles labels Feb 25, 2020
@mykeels mykeels self-assigned this Feb 25, 2020
@mykeels mykeels added this to PR in Progress in Simorgh via automation Feb 25, 2020
Copy link
Contributor

@simonsinclair simonsinclair left a comment

Choose a reason for hiding this comment

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

Wow, we've been struggling with this one! Well done. 👍
Bear with me though, I have quite a few questions:

@@ -538,53 +562,47 @@ exports[`Grid component should render Grid with Grid items including nested non-
@media (max-width:14.9375rem) {
.c1 {
width: calc(100% - 0%);
margin-left: 0%;
display: inline-block;
margin-left: 0% display:inline-block;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please can we verify this syntax? It looks invalid.

Comment on lines +81 to +84
@media (max-width:14.9375rem) {

}

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm seeing lots of additions in snapshots like this. Please can we confirm why they're appearing?

Comment on lines +11 to +14
<style
data-styled="active"
data-styled-version="5.0.1"
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm curious what this does - please can we confirm?

@@ -27,7 +27,8 @@
"snapshot"
],
"dependencies": {
"jest-styled-components": "^6.3.3"
"jest-styled-components": "^7.0.0",
"styled-components": "^5.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like a new dependency on styled-components has been added here. Please can you confirm why this has been added, when psammead-test-helpers doesn't use it directly? If it's because jest-styled-components complains, can it be a peer dependency?

@@ -3,6 +3,7 @@
<!-- prettier-ignore -->
| Version | Description |
|---------|-------------|
| 4.0.0 | [PR#3180](https://github.com/bbc/psammead/pull/3180) Bump styled components |
Copy link
Contributor

Choose a reason for hiding this comment

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

This change bumps jest-styled-components and adds styled-components as a dependency. This line should probably reflect that.

@mykeels
Copy link
Contributor Author

mykeels commented Feb 26, 2020

The snapshot issues are covered in this open issue.

@Bopchy Bopchy moved this from PR in Progress to Ready for dev in Simorgh Mar 3, 2020
@Bopchy Bopchy moved this from Ready for dev to Issue in Progress in Simorgh Mar 3, 2020
@Bopchy Bopchy moved this from Issue in Progress to PR in Progress in Simorgh Mar 5, 2020
Simorgh automation moved this from PR in Progress to Done Mar 24, 2020
@amywalkerdev
Copy link
Contributor

Closing this for now, waiting updates on jest styled components

@sareh sareh deleted the bump-styled-components branch March 26, 2020 15:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-articles Tasks for the WS Articles Team ws-home Tasks for the WS Home Team ws-media The World Service media stream
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Resolve circular dependency
4 participants