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

Use childNodes helper + Add source locations for some translation failures #1

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

ehaas
Copy link
Collaborator

@ehaas ehaas commented Nov 6, 2024

Just manual testing so far - do you have any thoughts on what our testing strategy should be?

@Vexu
Copy link
Member

Vexu commented Nov 6, 2024

Welcome!

My plan for testing was to construct TranslateC and CheckFile build steps from input test case files similar to how it is currently done in the Zig repo. That of course depends on a TranslateC build step which I started working on in https://github.com/ziglang/translate-c/tree/build-step but so far I've mostly been re familiarizing myself with the various parts of the build system.

Once that system works I plan on copying over all the current cases and adding something like // expect=fail to the non-passing cases to make it easy to track progress.

@BratishkaErik
Copy link

Hello, I have been lurking at ziglang organisation and noticed this new repo. First of all, thank you both (and squeek502 and TwoClocks and all other Aro contributors) for your work on that compiler, it is very cool project and I often enjoy reading new issues and code there :)

I guess this repo was not announced yet?

@Vexu
Copy link
Member

Vexu commented Nov 6, 2024

There's not much to announce yet but the intention of this repo is to implement ziglang/zig#20630

@Vexu Vexu merged commit c3dbba1 into main Nov 6, 2024
3 checks passed
@ehaas ehaas deleted the child-nodes branch November 6, 2024 20:58
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