Skip to content

Conversation

kant
Copy link
Contributor

@kant kant commented Oct 1, 2018

Closes IBM/carbon-components-angular#

Fixed typos on both strings 71 & 88

Changelog

New

  • Fixed typos

Changed

Removed

  • None

Copy link
Contributor

@cal-smith cal-smith 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 this!

Copy link
Member

@zvonimirfras zvonimirfras left a comment

Choose a reason for hiding this comment

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

Good catches, thanks!
Minor change, though.

- reference the related issue
- "closes #123" or "fixes #123" will close issue #123 once the PR is merged
- "issue #123" just refences the issue. Only use this if you definitly need the issue to remain open.
- "issue #123" just references the issue. Only use this if you definitively need the issue to remain open.
Copy link
Member

@zvonimirfras zvonimirfras Oct 1, 2018

Choose a reason for hiding this comment

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

definitly is actually meant to be an emphasis definitely

Suggested change
- "issue #123" just references the issue. Only use this if you definitively need the issue to remain open.
- "issue #123" just references the issue. Only use this if you definitely need the issue to remain open.

- relevant code snippets
- links to application source code or running demo ([Codesandbox is awesome for this!](https://codesandbox.io/s/angular)) (including connection/authentication information)
- add relevant labels (bug, accsibility, design, discussion, feature, etc)
- add relevant labels (bug, accesibility, design, discussion, feature, etc)
Copy link
Member

@zvonimirfras zvonimirfras Oct 1, 2018

Choose a reason for hiding this comment

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

Also, accessibility

Suggested change
- add relevant labels (bug, accesibility, design, discussion, feature, etc)
- add relevant labels (bug, accessibility, design, discussion, feature, etc)

@cal-smith
Copy link
Contributor

Hey! Just checking if you've had a chance to look at this recently? If not no worries, but we may close this out.

@carbon-bot
Copy link
Contributor

🎉 This issue has been resolved in version 2.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants