Skip to content

CLI, first class lang support: on demand common#2834

Merged
noise64 merged 21 commits intomainfrom
first-class-lang-on-demand-common
Mar 3, 2026
Merged

CLI, first class lang support: on demand common#2834
noise64 merged 21 commits intomainfrom
first-class-lang-on-demand-common

Conversation

@noise64
Copy link
Contributor

@noise64 noise64 commented Feb 24, 2026

@noise64 noise64 changed the title First class lang on demand common CLI, first class lang support: on demand common Feb 24, 2026
@noise64 noise64 added this to the Golem 1.5 milestone Feb 25, 2026
@noise64 noise64 self-assigned this Feb 25, 2026
@noise64 noise64 marked this pull request as ready for review February 25, 2026 18:30
@noise64 noise64 requested a review from vigoo February 25, 2026 19:29
@github-actions
Copy link

github-actions bot commented Feb 26, 2026

✅ All contributors have signed the CLA.
Posted by the CLA Assistant Lite bot.

github-actions bot added a commit that referenced this pull request Feb 26, 2026
@noise64
Copy link
Contributor Author

noise64 commented Feb 26, 2026

I have read the CLA Document and I hereby sign the CLA

static GOLEM_RUST_VERSION: &str = "2.0.0-dev.2";
static GOLEM_TS_VERSION: &str = "0.1.0-dev.1";
static GOLEM_AI_VERSION: &str = "v0.5.0-dev.1";
static GOLEM_AI_SUFFIX: &str = "-dev.wasm";
Copy link
Contributor

Choose a reason for hiding this comment

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

We no longer use the golem-ai wasms (just crate dependencies with versions)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i will clean them up, just wanted to check with you before: i guess we still should keep the env var hints for rust?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes those are still used in rust

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i'm cleaning them up in the next first-class-lang PR (and a few more common leftovers, which i found since then)

bail!(ContextInitHintError::CannotUseShortEnvRefWithLocalOrCloudFlags);
}

let sdk_overrides = SdkOverrides {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is anything changes regarding the overrides (anywhere) or this is just moving things around?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just moving for now

@noise64 noise64 merged commit c5b0f66 into main Mar 3, 2026
26 checks passed
@noise64 noise64 deleted the first-class-lang-on-demand-common branch March 3, 2026 16:49
@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Per-language embedded build files automatically extracted/updated into the app's temp directory

2 participants