-
Notifications
You must be signed in to change notification settings - Fork 54
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you also update the changelogs please 👍
sorry i wasn't clear earlier, i meant these files: https://github.com/bbc/psammead/blob/latest/packages/utilities/gel-foundations/CHANGELOG.md |
Updates changelogs for gel-foundations, psammead-assets, psammead-styles
@@ -376,7 +376,7 @@ export const sinhala = sinhalaStyles; | |||
export const somali = helmetFontStyles; | |||
export const swahili = helmetFontStyles; | |||
export const tamil = tamilStyles; | |||
export const telugu = teleguStyles; | |||
export const telugu = teluguStyles; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically isn't this actually a breaking change, therefore needing a major version change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
technically, yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh wait no it isn't because the export is ACTUALLY spelt correctly!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically, no. Because the export name did not change. That was always correct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahhhhhh 🙈 on that note, time for the weekend 🏃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Breaking changes needs major version change on psammead-styles
Resolves #1776
Overall change:
Fix Telugu typos in two packages
Code changes: