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

feat: Remove example app #2162

Merged
merged 5 commits into from
Jun 17, 2022
Merged

feat: Remove example app #2162

merged 5 commits into from
Jun 17, 2022

Conversation

haworku
Copy link
Contributor

@haworku haworku commented Jun 6, 2022

Summary

Remove /example and references to the example app. We use storybook for primary documentation and implementation examples.

Related Issues or PRs

#2001 some discussion here of the benefits of this change

How To Test

Library compiles and tests pass. No references to the example app remain that could confuse future contributors or maintainers.

@haworku haworku added the type: example Example implementations of ReactUSWDS, such as Storybook or kitchen sink app label Jun 6, 2022
@haworku haworku requested a review from a user June 6, 2022 16:39
@haworku haworku requested a review from rogeruiz as a code owner June 6, 2022 16:39
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2022

Warnings
⚠️

Changes were made to package.json, but not to yarn.lock - Perhaps you need to run yarn install?

Generated by 🚫 dangerJS against 43c00bc

@haworku haworku changed the title Remove example app feat: Remove example app Jun 6, 2022
@rogeruiz
Copy link
Contributor

rogeruiz commented Jun 8, 2022

I have to fix the Danger check here. It sees that package.json was modified, but it shouldn't be reporting changes being made to the contributors section.

@haworku
Copy link
Contributor Author

haworku commented Jun 9, 2022

@rogeruiz thanks for diagnosing

@rogeruiz
Copy link
Contributor

@haworku i fixed the Danger check in this PR #2167

@brandonlenz
Copy link
Contributor

@haworku i fixed the Danger check in this PR #2167

Just approved that one to hopefully help unblock this PR @rogeruiz

Copy link
Contributor

@rogeruiz rogeruiz left a comment

Choose a reason for hiding this comment

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

lgtm 🌈

@gidjin gidjin mentioned this pull request Jul 13, 2022
gidjin added a commit that referenced this pull request Jul 13, 2022
## [3.1.0](3.0.2...3.1.0) (2022-07-13)


### Features

* Remove example app ([#2162](#2162)) ([1eb579e](1eb579e))
* USWDS 2.13.3 ([#2175](#2175)) ([fdeefc6](fdeefc6))


### Bug Fixes

* Danger contributors check ([#2167](#2167)) ([c05fe06](c05fe06))
* Emit headingLevel type and do not emit type declarations for tests ([#2134](#2134)) ([a4501dd](a4501dd))
* export SummaryBoxHeading and SummaryBoxContent ([#2100](#2100)) ([141ba8e](141ba8e))


### Documentation & Examples

* Callout outside contributors process ([#1969](#1969)) ([7881191](7881191))
* Update releasing docs ([#2018](#2018)) ([6849dda](6849dda))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: example Example implementations of ReactUSWDS, such as Storybook or kitchen sink app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants