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

Move devdocs to package root, simplify scripts #397

Merged
merged 3 commits into from
Oct 23, 2018

Conversation

zetlen
Copy link
Contributor

@zetlen zetlen commented Oct 22, 2018

This PR is a:

[ ] New feature
[ ] Enhancement/Optimization
[ ] Refactor
[ ] Bugfix
[ ] Test for existing code
[ ] Documentation
[x] Other: organizational chore

Summary

When this pull request is merged, it will move devdocs to root to simplify scripts.

Confirmed with @jcalcaben that pwa-devdocs should not have its
dependencies versioned along with the rest of the Studio.

This helps us because it has external dependencies and defines a "build"
script, so every Lerna command had to specifically exclude it. Now our
Lerna scripts can be simpler.

Moved and audited package deps to compensate for change.

zetlen added 2 commits October 20, 2018 12:04
Confirmed with @jcalcabemn that pwa-devdocs should not have its
dependencies versioned along with the rest of the Studio.

This helps us because it has external dependencies and defines a "build"
script, so every Lerna command had to specifically exclude it. Now our
Lerna scripts can be simpler.

Moved and audited package deps to compensate for change.
@zetlen zetlen requested a review from jcalcaben October 22, 2018 12:26
@coveralls
Copy link

Coverage Status

Coverage remained the same at 16.105% when pulling 60fc7f1 on zetlen/move-devdocs-simplify-scripts into 2804e45 on release/2.0.

@zetlen zetlen merged commit cb19641 into release/2.0 Oct 23, 2018
@zetlen zetlen deleted the zetlen/move-devdocs-simplify-scripts branch October 23, 2018 16:45
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.

3 participants