-
Notifications
You must be signed in to change notification settings - Fork 137
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
ci: bump elastic stack version #1038
ci: bump elastic stack version #1038
Conversation
📦 Bundlesize report
|
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪 |
/test |
If this is related to an open issue, could you please link it to the issue number? If it's not, please just note that in the PR. Thanks! :) |
Done, in the description since there was no a tracked issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The scope args passing is incorrect which causes the saucelabs tests to fail.
SCOPE=SCOPE=@elastic/apm-rum-core
Might be from here -
Lines 560 to 566 in e1dcc58
def scopes = [ | |
'SCOPE=@elastic/apm-rum-core', | |
'SCOPE=@elastic/apm-rum', | |
'SCOPE=@elastic/apm-rum-react', | |
'SCOPE=@elastic/apm-rum-angular', | |
'SCOPE=@elastic/apm-rum-vue' | |
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
* ci: bump elastic stack version * chore: fix typo in the script * chore: set commit message * ci: remove scope prefix
What
Actions
Issues
There is no a tracked issue for this particular PR. I identified this particular need when raising a couple of issues regarding build failures in the CI.
Test
produces