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

Telugu typos #1847

Merged
merged 11 commits into from
Aug 23, 2019
Merged

Telugu typos #1847

merged 11 commits into from
Aug 23, 2019

Conversation

pharingee
Copy link
Contributor

@pharingee pharingee commented Aug 22, 2019

Resolves #1776

Overall change:
Fix Telugu typos in two packages

Code changes:

  • Fix typos in gel-foundations and spammed styles

  • 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

@pharingee pharingee self-assigned this Aug 22, 2019
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.

could you also update the changelogs please 👍

@pharingee pharingee requested a review from thekp August 22, 2019 11:53
@pharingee pharingee added this to the Psammead 2.0 milestone Aug 22, 2019
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;
Copy link
Contributor

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?

Copy link
Contributor

Choose a reason for hiding this comment

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

technically, yes

Copy link
Contributor

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!

Copy link
Contributor Author

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

Copy link
Contributor

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 🏃

Copy link
Contributor

@jamesbhobbs jamesbhobbs left a 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

@jamesbhobbs jamesbhobbs merged commit 9e0fa4c into latest Aug 23, 2019
@jamesbhobbs jamesbhobbs deleted the telugu-typos branch August 23, 2019 16:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix typo for Telugu in gel-foundations README
7 participants