Skip to content

Finish migrating regz tests#809

Merged
mattnite merged 1 commit intomainfrom
uncomment-regz-tests
Dec 30, 2025
Merged

Finish migrating regz tests#809
mattnite merged 1 commit intomainfrom
uncomment-regz-tests

Conversation

@mattnite
Copy link
Contributor

Back when I changed the filesystem layout of generated output, I commented out a bunch of tests, this patch finishes moving them over to the new system. Currently two fail so I'm keeping them commented out, and will fix after I've migrated regz to the new sqlite package (#808)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🔍 Lint Results

ℹ️ Additional issues on unchanged lines
The following 8 issue(s) exist but are not on lines changed in this PR:

tools/regz/src/gen.zig:421: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:526: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:679: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:729: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:829: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:1069: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:1071: TODO style comments need to have a linked microzig issue on the same line.
tools/regz/src/gen.zig:1152: TODO style comments need to have a linked microzig issue on the same line.

@mattnite mattnite force-pushed the uncomment-regz-tests branch from de136dc to 4125874 Compare December 30, 2025 18:20
@mattnite mattnite merged commit fe0877e into main Dec 30, 2025
48 checks passed
@mattnite mattnite deleted the uncomment-regz-tests branch December 30, 2025 18:27
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.

1 participant