Skip to content

Fix broken hoisting limits in yarn-plugin-validate-env#8633

Merged
tkajtoch merged 1 commit into
elastic:mainfrom
tkajtoch:build/fix-validate-env-hoisting-limits
Apr 25, 2025
Merged

Fix broken hoisting limits in yarn-plugin-validate-env#8633
tkajtoch merged 1 commit into
elastic:mainfrom
tkajtoch:build/fix-validate-env-hoisting-limits

Conversation

@tkajtoch
Copy link
Copy Markdown
Member

@tkajtoch tkajtoch commented Apr 25, 2025

Summary

This fixes hoisting limits for yarn-plugin-validate-env that made the release-cli build fail due to package version conflicts after merging #8450. yarn-plugin-validate-env is a whole separate utility package that isn't built often and can have hoisting limits set to workspaces.

QA

  • CI is passing for this PR
  • Checkout this PR locally, run rm -rf node_modules packages/*/node_modules && yarn and check if yarn workspace @elastic/eui-release-cli run build works as expected

@tkajtoch tkajtoch added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Apr 25, 2025
@tkajtoch tkajtoch self-assigned this Apr 25, 2025
@tkajtoch tkajtoch requested a review from a team as a code owner April 25, 2025 08:38
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

cc @tkajtoch

@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

cc @tkajtoch

Copy link
Copy Markdown
Contributor

@mgadewoll mgadewoll left a comment

Choose a reason for hiding this comment

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

🟢 LGTM! Checked it locally and the@elastic/eui-release-cli build script works as expected again 👍

@tkajtoch tkajtoch merged commit 688bb8d into elastic:main Apr 25, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants