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

Fix yarn resolutions on build #35854

Merged
merged 1 commit into from May 9, 2019

Conversation

Projects
None yet
4 participants
@mistic
Copy link
Member

commented May 1, 2019

Currently we have some problems with the resolutions during our build.

  • On master we are not removing the resolutions key from the package.json we ship.
  • On 7.x and 7.1 we declare resolutions in the main repo package.json but we do not copy them to the package.json into the build.

That PR intends to fix those problems.

@elasticmachine

This comment has been minimized.

Copy link

commented May 1, 2019

@elasticmachine

This comment has been minimized.

Copy link

commented May 1, 2019

@jbudz

jbudz approved these changes May 1, 2019

@mistic mistic removed the v7.0.2 label May 9, 2019

@mistic mistic merged commit 38a7ef1 into elastic:master May 9, 2019

2 checks passed

CLA All commits in pull request signed
Details
kibana-ci Build finished.
Details
@mistic

This comment has been minimized.

Copy link
Member Author

commented May 10, 2019

7.x: f448f06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.