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

add GitHub flavored markdown notes #2946

Merged
merged 13 commits into from
Feb 18, 2018

Conversation

swyxio
Copy link
Contributor

@swyxio swyxio commented Feb 8, 2018

@swyxio swyxio changed the title add GitHub flavored notes add GitHub flavored markdown notes Feb 8, 2018
@@ -3,18 +3,19 @@
[![Build Status on CircleCI](https://circleci.com/gh/storybooks/storybook.svg?style=shield)](https://circleci.com/gh/storybooks/storybook)
[![CodeFactor](https://www.codefactor.io/repository/github/storybooks/storybook/badge)](https://www.codefactor.io/repository/github/storybooks/storybook)
[![Known Vulnerabilities](https://snyk.io/test/github/storybooks/storybook/8f36abfd6697e58cd76df3526b52e4b9dc894847/badge.svg)](https://snyk.io/test/github/storybooks/storybook/8f36abfd6697e58cd76df3526b52e4b9dc894847)
[![BCH compliance](https://bettercodehub.com/edge/badge/storybooks/storybook)](https://bettercodehub.com/results/storybooks/storybook) [![codecov](https://codecov.io/gh/storybooks/storybook/branch/master/graph/badge.svg)](https://codecov.io/gh/storybooks/storybook)
[![BCH compliance](https://bettercodehub.com/edge/badge/storybooks/storybook)](https://bettercodehub.com/results/storybooks/storybook) [![codecov](https://codecov.io/gh/storybooks/storybook/branch/master/graph/badge.svg)](https://codecov.io/gh/storybooks/storybook)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

looks like my Prettier reformatted this README.md a bit, this can be ignored

Copy link
Member

Choose a reason for hiding this comment

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

Can you please revert stylistic changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reverted

</div>
~~~
`)(() => <Component />)
);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the relevant new documentation of the new exposed API. pretty simple.

@Hypnosphi
Copy link
Member

Please add a usage example to storybook-official

@Hypnosphi
Copy link
Member

You probably need to run yarn install in project root to update lockfile

@Hypnosphi
Copy link
Member

Live example

For some reason, it just displays raw markdown sources instead of formatted text

@Hypnosphi Hypnosphi self-assigned this Feb 18, 2018
@Hypnosphi
Copy link
Member

For some reason, it just displays raw markdown sources instead of formatted text

OK, it's because of the indentation in the example. Indent is just another way to define code block

@codecov
Copy link

codecov bot commented Feb 18, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@d15afc6). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #2946   +/-   ##
========================================
  Coverage          ?   92.5%           
========================================
  Files             ?       6           
  Lines             ?      40           
  Branches          ?       2           
========================================
  Hits              ?      37           
  Misses            ?       2           
  Partials          ?       1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d15afc6...216a8a6. Read the comment docs.

@Hypnosphi Hypnosphi merged commit fcbf6b2 into storybookjs:master Feb 18, 2018
@swyxio
Copy link
Contributor Author

swyxio commented Feb 19, 2018

ahh thats why. sorry I hadn't had time last week to get around to this, glad you figured it out. was definitely working in my local setup. im still in the first month of my new job, got to figure out what my ability is to work on open source :( thank you so much for being an awesome maintainer, I hope to be a better contributor over time

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

Successfully merging this pull request may close these issues.

3 participants