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

Commonjs #117

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Commonjs #117

wants to merge 21 commits into from

Conversation

hthetiot
Copy link
Contributor

No description provided.

@hthetiot hthetiot added the wip label Jul 18, 2017
@hthetiot hthetiot added this to the v18.0.0 CommonJS milestone Nov 30, 2017
@cdebost cdebost force-pushed the commonjs branch 12 times, most recently from 8d429ec to 0fe8d64 Compare January 25, 2019 01:28
@cdebost
Copy link
Collaborator

cdebost commented Jan 31, 2019

Although the commonjs branches of montage, mr, and mop are passing all suites in travis, I'm seeing a really strange bug when mopping contour-app. At this point there are too many changes and refactors being done at once in the commonjs PRs to evaluate what's going on and understand what is really being changed. I've chosen to break out the npm3-specific changes into standalone feature/npm3 PRs for each repo. We can get those reviewed and merged as the basis of 18.0.0. Any non-breaking changes related to commonjs from this PR can be applied over 18's lifetime, saving any breaking changes for 19.0.0.

npm3 changes for mr are here: #145.

@cdebost cdebost changed the title Commonjs and npm3+ support Commonjs Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants