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

Deploy codesandbox example to gh-pages #59

Merged
merged 4 commits into from May 26, 2019

Conversation

istarkov
Copy link
Contributor

No description provided.

@istarkov istarkov requested a review from TrySound May 26, 2019 09:08
@istarkov istarkov self-assigned this May 26, 2019
package.json Outdated
@@ -17,7 +17,9 @@
"jest": "NODE_ICU_DATA=`yarn -s run node-full-icu-path` jest",
"test": "eslint ./ && flow check && yarn test:ts && yarn jest",
"test:ts": "tslint -p tsconfig.json",
"deploy": "yarn next build && yarn next export && gh-pages -d out",
"copy-sandbox-examples": "mkdir -p ./out/pages && find ./pages -type d -mindepth 1 -maxdepth 1 -exec cp -R \\{\\} ./out/pages \\;",
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe rename to codesandboxes then?

Copy link
Contributor

Choose a reason for hiding this comment

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

I meant rename dist folder to codesandboxes. pages doesn't make much sense in gh-pages

Copy link
Contributor Author

Choose a reason for hiding this comment

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

:-) done

next.config.js Outdated
const rifmPath =
process.env.NODE_ENV === 'production'
? './dist/rifm.esm.js'
: './src/index.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

Does source work without next babel config extension?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep it works, even reloads on change

Copy link
Contributor

Choose a reason for hiding this comment

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

Then let's consume only src. I used dist to consume released version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@istarkov istarkov merged commit b529921 into master May 26, 2019
@istarkov istarkov deleted the deploy-codesandbox-2-gh-pages branch May 26, 2019 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants