Skip to content

Color -> ANSI for chat in terminal#3224

Open
Wunka wants to merge 23 commits into
PixelGuys:masterfrom
Wunka:ANSI
Open

Color -> ANSI for chat in terminal#3224
Wunka wants to merge 23 commits into
PixelGuys:masterfrom
Wunka:ANSI

Conversation

@Wunka

@Wunka Wunka commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Finnally printing ANSI inside the terminal

@Wunka Wunka marked this pull request as ready for review June 13, 2026 15:09
@Wunka Wunka moved this to Low Priority in PRs to review Jun 13, 2026
@Wunka

Wunka commented Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

I think this is ready. I don't know where to put he function. Before I tried to do it directly on log but the stackallocator is not always there when logging, thats why its currently in main.zig

Comment thread src/main.zig Outdated
Comment thread src/server/server.zig Outdated
@IntegratedQuantum IntegratedQuantum moved this from Low Priority to In review in PRs to review Jun 14, 2026
Comment thread src/log.zig Outdated
Comment thread src/log.zig Outdated
Comment thread src/server/server.zig Outdated
Wunka and others added 4 commits June 15, 2026 13:54
Co-authored-by: IntegratedQuantum <43880493+IntegratedQuantum@users.noreply.github.com>
Comment thread src/main.zig Outdated
Comment thread src/main.zig Outdated
runtimeArgs[i] = .fromAnytype(@TypeOf(args[i]), &args[i]);
}
runtimeLogFn(level, format, &runtimeArgs);
log.runtimeLogFn(@enumFromInt(@intFromEnum(level)), format, &runtimeArgs);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this is a good idea, please use a switch statement or a string.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

all done

Comment thread src/log.zig Outdated
Comment thread src/log.zig Outdated
Comment thread src/log.zig Outdated
Comment thread src/log.zig Outdated
Wunka and others added 4 commits June 15, 2026 17:12
Co-authored-by: IntegratedQuantum <43880493+IntegratedQuantum@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants