build: add test-mach step to test src/ and test for testing all sub-projects #422

Merged
alichraghi merged 3 commits from root-build into main 2022-07-19 16:25:55 +00:00
alichraghi commented 2022-07-19 10:06:59 +00:00 (Migrated from github.com)

Fixes #396

  • By selecting this checkbox, I agree to license my contributions to this project under the license(s) described in the LICENSE file, and I have the right to do so or have received permission to do so by an employer or client I am producing work for whom has this right.
Fixes #396 - [x] By selecting this checkbox, I agree to license my contributions to this project under the license(s) described in the LICENSE file, and I have the right to do so or have received permission to do so by an employer or client I am producing work for whom has this right.
emidoots (Migrated from github.com) reviewed 2022-07-19 16:22:52 +00:00
@ -12,3 +9,4 @@
return b.addTest(thisDir() ++ "/src/main.zig");
}
pub const pkg = std.build.Pkg{
emidoots (Migrated from github.com) commented 2022-07-19 16:22:51 +00:00

I don't see setBuildMode being called anymore here, so this means we cannot test in release/debug modes easily now, right? I think we should keep that if possible

I don't see `setBuildMode` being called anymore here, so this means we cannot test in release/debug modes easily now, right? I think we should keep that if possible
emidoots (Migrated from github.com) reviewed 2022-07-19 16:24:03 +00:00
@ -12,3 +9,4 @@
return b.addTest(thisDir() ++ "/src/main.zig");
}
pub const pkg = std.build.Pkg{
emidoots (Migrated from github.com) commented 2022-07-19 16:24:03 +00:00

I won't block this PR on this, but we should fix that.

I won't block this PR on this, but we should fix that.
emidoots (Migrated from github.com) approved these changes 2022-07-19 16:24:06 +00:00
alichraghi (Migrated from github.com) reviewed 2022-07-19 16:25:42 +00:00
@ -12,3 +9,4 @@
return b.addTest(thisDir() ++ "/src/main.zig");
}
pub const pkg = std.build.Pkg{
alichraghi (Migrated from github.com) commented 2022-07-19 16:25:42 +00:00

will, it's possible but then we have to pass/accept to every testStep functions
i'll fix this in next incoming PR

will, it's possible but then we have to pass/accept to every `testStep` functions i'll fix this in next incoming PR
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
hexops/mach!422
No description provided.