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

Attempt to fix broken envs during build #11584

Merged
merged 13 commits into from
Nov 21, 2024
Merged

Conversation

Frizi
Copy link
Contributor

@Frizi Frizi commented Nov 18, 2024

Pull Request Description

Attempt to fix build env issues causing "Enso undefined" version and missing AG grid license key.

Screenshot taken from a CI build:
image

@Frizi Frizi added the CI: No changelog needed Do not require a changelog entry for this PR. label Nov 18, 2024
app/common/src/appConfig.js Show resolved Hide resolved
@@ -6,4 +6,4 @@ required-versions:
# NB. The Rust version is pinned in rust-toolchain.toml.
# NB. The Node version is pinned in .node-version.
cargo-watch: ^8.1.1
wasm-pack: ^0.12.1
wasm-pack: ">=0.12.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this change? Are not afraid of wasm-pack updates even a bit?

Copy link
Contributor Author

@Frizi Frizi Nov 21, 2024

Choose a reason for hiding this comment

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

I am using 0.13 and there is no issue with that. Historically they also didn't really make much updates. 0.13 was released a few weeks ago and it was a first release since a few years back. For CI, we pin the installed version anyway. This change just makes the potential compatibility issues with local setup go away, as any new installation will default to 0.13 now. We are phasing out wasm-pack usage completely soon enough anyway.

@MrFlashAccount MrFlashAccount added CI: Keep up to date Automatically update this PR to the latest develop. CI: Clean build required CI runners will be cleaned before and after this PR is built. labels Nov 21, 2024
@MrFlashAccount
Copy link
Contributor

MrFlashAccount commented Nov 21, 2024

I added the labels above as possible solution for failed storybook deploy task: https://github.com/enso-org/enso/actions/runs/11953657884/job/33322022010?pr=11584

For some reason it doesn't detect the chromatic config which happen on some PRs but doesn't on the others

GitHub
Enso Analytics is a self-service data prep and analysis platform designed for data teams. - Attempt to fix broken envs during build · 295e155

Copy link

github-actions bot commented Nov 21, 2024

🧪 Storybook is successfully deployed!

📊 Dashboard:

@Frizi Frizi added CI: Ready to merge This PR is eligible for automatic merge and removed CI: Keep up to date Automatically update this PR to the latest develop. CI: Clean build required CI runners will be cleaned before and after this PR is built. labels Nov 21, 2024
@jdunkerley jdunkerley merged commit 0e31169 into develop Nov 21, 2024
51 of 54 checks passed
@jdunkerley jdunkerley deleted the wip/frizi/undefined-version branch November 21, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants