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

Merge 0.4.x, excluding #1113 #1116

Merged
merged 23 commits into from
May 31, 2023
Merged

Merge 0.4.x, excluding #1113 #1116

merged 23 commits into from
May 31, 2023

Conversation

pitdicker
Copy link
Collaborator

The changes are small but I wanted to do it now because I don't know if excluding #1113 will remain easy.

pitdicker and others added 23 commits May 27, 2023 00:12
Force the `run` script to fail early using bash shell
`set` options.

Previously, errors before the last statement of the script would be
ignored.
The script at the URL uses bash-isms. Previously `sh` was silently emitting
an error but by luck was succeeding.
NFC fix most `yamllint` errors and warnings in
`lint.yml` and `test.yml`.
Sanity check `date` command used in tests `try_verify_against_date_command`
and `try_verify_against_date_command_format`. Make sure it's the kind of
`data` command expected (accepts `--version`).
Also, it's just good to print this for debugging purposes.
Print progress messages for human users watching this long-running
test `try_verify_against_date_command`.
@djc djc merged commit 088b693 into chronotope:main May 31, 2023
33 checks passed
@pitdicker pitdicker deleted the merge_0.4.x branch May 31, 2023 11:03
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.

None yet

6 participants