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

Update psammead snapshots #783

Merged
merged 20 commits into from
Jul 9, 2019
Merged

Update psammead snapshots #783

merged 20 commits into from
Jul 9, 2019

Conversation

jroebu14
Copy link
Contributor

@jroebu14 jroebu14 commented Jul 5, 2019

Resolves #276

Overall change: Updates all Psammead's packages to the latest version of psammead-test-helpers to make use of new snapshot generator helper that now uses @testing-library/react.

Code changes:

  • any package.json files containing psammead-test-helpers upgraded to 1.0.1
  • package-lock.json files regenerated
  • ran updateSnapshots script to regenerate snapshot files

  • I have assigned myself to this PR and the corresponding issues
  • Tests added for new features
  • Test engineer approval

Testing notes
These changes should only affect our test snapshots and nothing user-facing. The risk of regression is low so probably none to minimal testing required.

@jroebu14 jroebu14 self-assigned this Jul 5, 2019
@jroebu14 jroebu14 marked this pull request as ready for review July 5, 2019 09:28
@thekp
Copy link
Contributor

thekp commented Jul 5, 2019

i think for each package you've updated, you will have to update the respective package.json version and update the changelog. so that it can be republished on NPM.
😭

Jonathan Roebuck added 2 commits July 8, 2019 09:21
Bump package versions for psammead-brand, psammead-caption, psammead-consent-banner, psammead-copyright, psammead-figure, psammead-headings, psammead-image-placeholder, psammead-image, psammead-inline-link, psammead-media-indicator, psammead-navigation, psammead-paragraph, psammead-section-label, psammead-sitewide-links, psammead-story-promo-list, psammead-story-promo, psammead-timestamp, psammead-visually-hidden-text, psammead-timestamp-container, gel-foundations, psammead-assets, psammead-storybook-helpers, psammead-styles
Updates changelogs for psammead-brand, psammead-caption, psammead-consent-banner, psammead-copyright, psammead-figure, psammead-headings, psammead-image-placeholder, psammead-image, psammead-inline-link, psammead-media-indicator, psammead-navigation, psammead-paragraph, psammead-section-label, psammead-sitewide-links, psammead-story-promo-list, psammead-story-promo, psammead-timestamp, psammead-visually-hidden-text, psammead-timestamp-container, gel-foundations, psammead-assets, psammead-storybook-helpers, psammead-styles
@jroebu14 jroebu14 force-pushed the update-psammead-snapshots branch from c79e25a to 595ca87 Compare July 8, 2019 08:28
Copy link
Contributor

@thekp thekp left a comment

Choose a reason for hiding this comment

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

I believe there are 2 missing CHANGELOG updates:

psammead-images
psammead-story-promo

edit: the files were changed and github did not show the changed files 🙃

@jroebu14
Copy link
Contributor Author

jroebu14 commented Jul 8, 2019

@thekp whoa good catch!

@jroebu14 jroebu14 force-pushed the update-psammead-snapshots branch from f4c82bf to 595ca87 Compare July 8, 2019 10:22
Copy link
Contributor

@thekp thekp left a comment

Choose a reason for hiding this comment

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

thanks for the updates, nice work 💯

Copy link
Contributor

@dr3 dr3 left a comment

Choose a reason for hiding this comment

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

psammead-image needs a changelog, otherwise LGTM

Also this PR has ALLLLL the conflicts

Jonathan Roebuck added 3 commits July 9, 2019 08:33
Bump package versions for psammead-brand, psammead-caption, psammead-consent-banner, psammead-copyright, psammead-figure, psammead-headings, psammead-image-placeholder, psammead-image, psammead-inline-link, psammead-media-indicator, psammead-navigation, psammead-paragraph, psammead-section-label, psammead-sitewide-links, psammead-story-promo-list, psammead-story-promo, psammead-timestamp, psammead-visually-hidden-text, psammead-timestamp-container, gel-foundations, psammead-assets, psammead-storybook-helpers, psammead-styles
Updates changelogs for psammead-brand, psammead-caption, psammead-consent-banner, psammead-copyright, psammead-figure, psammead-headings, psammead-image-placeholder, psammead-image, psammead-inline-link, psammead-media-indicator, psammead-navigation, psammead-paragraph, psammead-section-label, psammead-sitewide-links, psammead-story-promo-list, psammead-story-promo, psammead-timestamp, psammead-visually-hidden-text, psammead-timestamp-container, gel-foundations, psammead-assets, psammead-storybook-helpers, psammead-styles
@jroebu14 jroebu14 force-pushed the update-psammead-snapshots branch from 2ec9299 to c61b3df Compare July 9, 2019 08:19
@PriyaKR
Copy link
Contributor

PriyaKR commented Jul 9, 2019

Doesn't need a testing from a tester.

@jroebu14 jroebu14 merged commit 14d4120 into latest Jul 9, 2019
@jroebu14 jroebu14 deleted the update-psammead-snapshots branch July 9, 2019 10:07
@jroebu14 jroebu14 mentioned this pull request Jul 10, 2019
3 tasks
This was referenced Jul 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Psammead snapshots (react-testing-library)
5 participants