Age | Commit message (Collapse) | Author |
|
More improvements to build.zig and test/tests.zig
|
|
Commit dbd42bb (Cleaning up zig build output) broke the unit test.
Always use exit code 2, instead of 0. This is the exit code used by the
build runner to notify the compiler to not report any further
diagnostics.
Move the Ziglings logo from the `build` function scope to the global
scope, and make it public so that tests.zig can use it to find the
number of lines to skip, instead of using an hard coded value.
Fixes #295
|
|
Use fs.path.sep_str instead of a slash, in literal paths.
|
|
Use directly the RunStep.addCheck method, instead.
|
|
Avoid too long lines or too many line breaks.
|
|
Use it in Zigling.compile, in order to reduce code duplication.
|
|
Some functions and custom build steps incorrectly used a normal comment.
Use a doc-comment instead.
Additionally, use a present tense verb to describe the action of a
function or custom build step.
|
|
Instead of marking a parameter as unused inside the function body.
|
|
Update PrintStep and SkipStep to use the `self` variable when getting
the parent pointer from Step.
This convention is used in `std.Build`.
|
|
Update the code in `zig build install` and `zig build -Dn=n install`, so
that exercises that must be skipped are not installed, since it will
cause an error. Ensure that a skip message is printed.
|
|
The command fails because the path to the exercises directory was
incorrectly set to "exercises" instead of `work_path`.
The bug was introduced in commit b56bb7b
(build: enable full parallelism when -Dhealed is set).
Remove the comment about not using multi-object loop, since it is
confusing.
|
|
|
|
* Only show actual Zig compiler errors, not build internals to
confused and dismay.
* Remove advanced usage instructions not needed in normal
operation.
|
|
|
|
Add missing word "a"
|
|
Improve wording in some comments
|
|
Fix reference to slice_ptr in example code
|
|
Remove repeated word "with"
|
|
Add missing word "of"
|
|
Add missing word "to"
|
|
fix typo well -> will
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Improve instructions.
|
|
|
|
|
|
|
|
Improve Exercise type more
|
|
Replace the file_path variable with path.
|
|
Update the output and hint fields in the `exercises` slice to use a
multiline string literal when the string have multiple lines or use the
`"` character. This will greatly improve readability.
Additionally, remove the trailing whitespace on each line and check it
in the validate_exercises function.
Update the output comparison logic in ZiglingStep, since the current
code assumes that the string has only one line.
Update test/tests.zig to use the new `CheckNamedStep` in test case 1,
since RunStep.StdIo.Check is no longer able to correctly check the
output.
Fixes #283
|
|
|
|
Use `orelse unreachable` instead, in order to simplify the code.
Fix doc-comments in the Exercise type.
|
|
Use an optional type, instead of an empty string, since it is more
idiomatic.
|
|
|
|
Add the -Dhealed-path option
|
|
This is necessary in the unit tests, to ensure each test case use a
different exercises directory.
Update test/tests.zig to use the new healed-path option, ensuring that
each temp directory is removed.
In test case 3, 4 and 5, move case_step as the first step in the
dependency chain. This will improve the build summary tree.
In test case 5, remove the dependency to heal_step, since it is not
necessary.
|
|
Fixed example syntax for inline else
|
|
|
|
|
|
|
|
Remove the comment about using POSIX patch variant, since we now use the
-s option.
|
|
CheckStep, FailStep, fail, HealStep and heal incorrectly used a normal
comment, instead of a doc-comment.
Additionally, improve the documentation for FailStep and HealStep.
|
|
In test case 1 and 2, remove the `i` variable and use `ex.number()`
instead.
In test case 2, when checking the exercise output from stderr, also
check that stdout is empty and vice versa.
|
|
Clarify the methods syntax sugar & a bit more
|