Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix company name for percy.io #3934

Merged
merged 2 commits into from
Jul 31, 2018
Merged

Fix company name for percy.io #3934

merged 2 commits into from
Jul 31, 2018

Conversation

faun
Copy link
Contributor

@faun faun commented Jul 30, 2018

Change Perci => Percy company name for Percy.io

Issue:

What I did

Change company name from Perci to Percy

How to test

View the markdown and confirm the change is correct.

Is this testable with Jest or Chromatic screenshots?

I suppose it's possible, but I imagine this has already been covered. Let's run the tests and find out if anything changes.

Does this need a new example in the kitchen sink apps?

Nope.

Does this need an update to the documentation?

This is the documentation.

Change Perci => Percy company name for [Percy.io](https:percy.io)
@danielduan danielduan added documentation patch:yes Bugfix & documentation PR that need to be picked to main branch labels Jul 30, 2018
Copy link
Member

@wuweiweiwu wuweiweiwu left a comment

Choose a reason for hiding this comment

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

LGTM

@wuweiweiwu wuweiweiwu merged commit 7a2c5bc into storybookjs:master Jul 31, 2018
@Hypnosphi Hypnosphi added the patch:done Patch/release PRs already cherry-picked to main/release branch label Aug 2, 2018
Hypnosphi pushed a commit that referenced this pull request Aug 2, 2018
Fix company name for percy.io
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation patch:done Patch/release PRs already cherry-picked to main/release branch patch:yes Bugfix & documentation PR that need to be picked to main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants