Skip to content

Release v0.2.4 for Bazel Central Registry.#354

Open
PiotrSikora wants to merge 1 commit intoproxy-wasm:mainfrom
PiotrSikora:releng_v0.2.4-bazel
Open

Release v0.2.4 for Bazel Central Registry.#354
PiotrSikora wants to merge 1 commit intoproxy-wasm:mainfrom
PiotrSikora:releng_v0.2.4-bazel

Conversation

@PiotrSikora
Copy link
Member

No description provided.

@PiotrSikora PiotrSikora changed the base branch from main to bazel February 4, 2026 20:42
@PiotrSikora PiotrSikora changed the title Re-release v0.2.4 for Bazel Central Registry. Release v0.2.4 for Bazel Central Registry. Feb 4, 2026
@PiotrSikora
Copy link
Member Author

PiotrSikora commented Feb 4, 2026

I've decided to tag v0.2.4-bazel off branch, since there are no code changes in the SDK since v0.2.4, so that we can push this as a official v0.2.4 release to BCR.

Let me know if that works, or if you prefer to wait until we release v0.2.5.

cc @mmorel-35 @phlax

@PiotrSikora PiotrSikora changed the base branch from bazel to main February 4, 2026 20:58
@PiotrSikora PiotrSikora changed the base branch from main to bazel/v0.2.4 February 4, 2026 20:58
@PiotrSikora
Copy link
Member Author

Please ignore the outdated check, since it's only for a dependency update in one of the examples, and we don't need (or want, really, since it would diverge from tagged v0.2.4 release) to fix it before merging this.

@phlax
Copy link

phlax commented Feb 4, 2026

fine from Envoy pov - main is still on workspace, and im using a registry hosted in the toolshed repo atm - ill test the bcr version as soon as i get chance

@mmorel-35
Copy link
Contributor

@phlax ,
I saw bazelbuild/bazel-central-registry#7229

You've use dashed while there are underscore in the module name.

That's what I initially provided but @PiotrSikora preferred underscores if I remember correctly

@PiotrSikora
Copy link
Member Author

PiotrSikora commented Feb 5, 2026

That's what I initially provided but @PiotrSikora preferred underscores if I remember correctly

AFAICT, you provided underscores from the first commit in #333.

And no, that's definitely not my preference...

But it matches the underscored name is our in WORKSPACE file, which uses dashes, because as far as I recall, you couldn't use dashes in the workspace names (at least not originally...), but it seems that restriction doesn't apply to Bazel modules, so perhaps we should use dashes again?

Note that if you look at the @load statements in .bzl files (here and in Envoy), then you'll notice that all external workspaces are using underscores instead of dashes in their names.

@phlax
Copy link

phlax commented Feb 5, 2026

i think my preference is towards using kebab-case for mod names - it kinda makes it clearer that they are names

@PiotrSikora PiotrSikora changed the base branch from bazel/v0.2.4 to main February 6, 2026 21:31
@PiotrSikora
Copy link
Member Author

I've decided to tag v0.2.4-bazel off branch, since there are no code changes in the SDK since v0.2.4, so that we can push this as a official v0.2.4 release to BCR.

Actually, since there are no code changes, let's merge it to main branch, and then revert version back to v0.2.5-dev.

Signed-off-by: Piotr Sikora <code@piotrsikora.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants