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

Bump psammead-media-player #1920

Merged
merged 3 commits into from
Aug 27, 2019
Merged

Bump psammead-media-player #1920

merged 3 commits into from
Aug 27, 2019

Conversation

ghost
Copy link

@ghost ghost commented Aug 27, 2019

Resolves #NUMBER

Overall change: Bump version missed on #1919

Code changes:

  • Bump version and update Changelog

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

@ghost ghost added ws-home Tasks for the WS Home Team simorgh-core-stream labels Aug 27, 2019
@ghost ghost self-assigned this Aug 27, 2019
@ghost ghost marked this pull request as ready for review August 27, 2019 12:58
@thekp
Copy link
Contributor

thekp commented Aug 27, 2019

https://github.com/bbc/psammead/pull/1919/files#diff-f9a34775b213d576f538360e3242a9daR6

Should we update the text in the changelog here, to like "version bump", since it is misleading?

I don't think you will need to update the package's version again since it's just a changelog.

@ghost
Copy link
Author

ghost commented Aug 27, 2019

@thekp I will update it in #1917

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.

I personally would prefer you fix the test helper changelog in this PR, as its much cleaner, but as youve said you will do it in another PR, ile let it slide

@FK78 FK78 merged commit 0f214ed into latest Aug 27, 2019
@FK78 FK78 deleted the fix-package-update branch August 27, 2019 15:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
simorgh-core-stream ws-home Tasks for the WS Home Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants