Skip to content

fix(broccoli): Rebuild diffs input changes and output patch separately. #197

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

Merged
merged 7 commits into from
Sep 13, 2018

Conversation

amiller-gh
Copy link
Contributor

Fixes a bug where changes are occasionally not picked up.

@amiller-gh amiller-gh force-pushed the broccoli-rebuild-fix branch 3 times, most recently from eb33c53 to 7548289 Compare September 11, 2018 19:16
@amiller-gh
Copy link
Contributor Author

amiller-gh commented Sep 11, 2018

@chriseppstein this makes CI green and fixes a pretty big bug with the ember-cli addon. Once in I'll release 0.20.0-beta.1 👍

Copy link
Contributor

@chriseppstein chriseppstein left a comment

Choose a reason for hiding this comment

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

The description of this PR says it fixes a bug relating to caching, but I don't see any new test that would fail without this code change.

@chriseppstein
Copy link
Contributor

Approved pending the addition of test(s) for the primary issue this addresses.

@amiller-gh amiller-gh merged commit 659ae68 into master Sep 13, 2018
@amiller-gh amiller-gh deleted the broccoli-rebuild-fix branch September 13, 2018 19:11
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.

2 participants