Skip to content

Comments

[zig] Update zap to 0.11#10805

Open
dragosv wants to merge 3 commits intoTechEmpower:masterfrom
dragosv:zap_015
Open

[zig] Update zap to 0.11#10805
dragosv wants to merge 3 commits intoTechEmpower:masterfrom
dragosv:zap_015

Conversation

@dragosv
Copy link
Contributor

@dragosv dragosv commented Feb 21, 2026

Updated zap to 0.11 and zig to 0.15

@joanhey
Copy link
Contributor

joanhey commented Feb 22, 2026

Please add yourself to the maintainers in the benchmark_config.json.

So with any change or marked as broken, you will be notified.

Example:

{
"framework": "quarkus",
"maintainers": ["franz1981", "Sanne", "geoand"],
"tests": [
{
"default": {

Comment on lines +21 to +23
RUN tar -xvf zig-$(uname -m)-linux-${ZIG_VER}.tar.xz

RUN mv zig-$(uname -m)-linux-${ZIG_VER} /usr/local/zig
Copy link
Contributor

@joanhey joanhey Feb 22, 2026

Choose a reason for hiding this comment

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

Please do it in one line.

RUN tar -xvf zig-$(uname -m)-linux-${ZIG_VER}.tar.xz /usr/local/zig

For faster builds.
And try to use a quiet option to be less verbose.
Thank you.

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.

2 participants