Skip to content

Commit

Permalink
Remove the test-cra example app.
Browse files Browse the repository at this point in the history
No longer needed, we can now test `cra-kitchen-sink`
  • Loading branch information
tmeasday committed Aug 22, 2017
1 parent 8f4d05a commit e27a9c0
Show file tree
Hide file tree
Showing 21 changed files with 3 additions and 232 deletions.
23 changes: 0 additions & 23 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,28 +72,6 @@ jobs:
name: "Build vue kitchen-sink"
command: |
cd examples/vue-kitchen-sink && npm run build-storybook
example-test-cra:
<<: *defaults
steps:
- checkout
- restore_cache:
keys:
- root-dependencies-{{ checksum "package.json" }}
- root-dependencies-
- run:
name: "Install root dependencies"
command: |
yarn install
- run:
name: "Bootstrapping"
command: |
npm run bootstrap
npm run build-packs
npm run bootstrap:test-cra
- run:
name: "Build test-cra"
command: |
cd examples/test-cra && npm run build-storybook
example-react-native:
<<: *defaults
steps:
Expand Down Expand Up @@ -189,7 +167,6 @@ workflows:
- validate
- build
- example-kitchen-sinks
- example-test-cra
- example-react-native
- docs
- lint
Expand Down
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ To test your project against the current latest version of storybook, you can cl

The best way to help figure out an issue you are having is to produce a minimal reproduction against the `master` branch.

A good way to do that is using the example `test-cra` app embedded in this repository:
A good way to do that is using the example `cra-kitchen-sink` app embedded in this repository:

```bash
# Download and build this repository:
Expand All @@ -50,7 +50,7 @@ cd storybook
npm install
npm run bootstrap

cd examples/test-cra
cd examples/cra-kitchen-sink

# make changes to try and reproduce the problem, such as adding components + stories
npm start storybook
Expand Down
18 changes: 0 additions & 18 deletions examples/test-cra/.gitignore

This file was deleted.

3 changes: 0 additions & 3 deletions examples/test-cra/.storybook/addons.js

This file was deleted.

10 changes: 0 additions & 10 deletions examples/test-cra/.storybook/config.js

This file was deleted.

1 change: 0 additions & 1 deletion examples/test-cra/README.md

This file was deleted.

33 changes: 0 additions & 33 deletions examples/test-cra/package.json

This file was deleted.

Binary file removed examples/test-cra/public/favicon.ico
Binary file not shown.
31 changes: 0 additions & 31 deletions examples/test-cra/public/index.html

This file was deleted.

24 changes: 0 additions & 24 deletions examples/test-cra/src/App.css

This file was deleted.

16 changes: 0 additions & 16 deletions examples/test-cra/src/App.js

This file was deleted.

10 changes: 0 additions & 10 deletions examples/test-cra/src/App.test.js

This file was deleted.

5 changes: 0 additions & 5 deletions examples/test-cra/src/index.css

This file was deleted.

8 changes: 0 additions & 8 deletions examples/test-cra/src/index.js

This file was deleted.

7 changes: 0 additions & 7 deletions examples/test-cra/src/logo.svg

This file was deleted.

20 changes: 0 additions & 20 deletions examples/test-cra/src/stories/ComponentWithRef.js

This file was deleted.

16 changes: 0 additions & 16 deletions examples/test-cra/src/stories/index.js

This file was deleted.

1 change: 0 additions & 1 deletion jest.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ module.exports = {
'<rootDir>/app',
'<rootDir>/lib',
'<rootDir>/examples/cra-kitchen-sink',
'<rootDir>/examples/test-cra',
],
testPathIgnorePatterns: ['/node_modules/'],
collectCoverage: false,
Expand Down
2 changes: 0 additions & 2 deletions lerna.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,13 @@
"bootstrap": {
"ignore": [
"crna-kitchen-sink",
"test-cra",
"react-native-vanilla"
]
},
"publish": {
"ignore": [
"cra-kitchen-sink",
"crna-kitchen-sink",
"test-cra",
"react-native-vanilla",
"vue-example"
]
Expand Down
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
"bootstrap:docs": "cd docs && yarn install",
"bootstrap:react-native-vanilla": "lerna exec --scope react-native-vanilla -- npm install",
"bootstrap:crna-kitchen-sink": "lerna exec --scope crna-kitchen-sink -- npm install",
"bootstrap:test-cra": "lerna exec --scope test-cra -- npm install",
"build-packs": "lerna exec --scope '@storybook/*' --parallel -- ../../scripts/build-pack.sh ../../packs",
"start": "lerna run --stream --scope cra-kitchen-sink storybook",
"changelog": "pr-log --sloppy",
Expand Down
2 changes: 1 addition & 1 deletion packs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@

This directory is filled with git packs of our packages when you run the `build-packs` command.

The purpose of this is to enable the `test-cra` app to use the packs as "local" installs without linking, and thus behave like a normal app would, yet still use our latest code. It's awkward for development but good to double check things, especially in CI.
The purpose of this is to enable the example apps to use the packs as "local" installs without linking, and thus behave like a normal app would, yet still use our latest code. It's awkward for development but good to double check things, especially in CI.

0 comments on commit e27a9c0

Please sign in to comment.