Skip to content

CI: simplify for readability #118

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

Merged
merged 1 commit into from
Mar 18, 2021
Merged

CI: simplify for readability #118

merged 1 commit into from
Mar 18, 2021

Conversation

ojeda
Copy link
Member

@ojeda ojeda commented Mar 18, 2021

  • Use YAML | blocks more, since they need less escaping,
    i.e. they have easier rules to follow.

  • Group several run keys into one block to reduce the number
    of steps in the web interface.

  • Fix the output of QEMU so that the following test regex are easier to
    read; plus it enables proper checking of the end of the line with $.

  • Use grep's BREs for all, they are easier to read.

Signed-off-by: Miguel Ojeda [email protected]

  - Use YAML | blocks more, since they need less escaping,
    i.e. they have easier rules to follow.

  - Group several `run` keys into one block to reduce the number
    of steps in the web interface.

  - Fix the output of QEMU so that the following test regex are easier to
    read; plus it enables proper checking of the end of the line with $.

  - Use grep's BREs for all, they are easier to read.

Signed-off-by: Miguel Ojeda <[email protected]>
@alex alex merged commit 9bc557e into rust Mar 18, 2021
@alex alex deleted the ci-simplify branch March 18, 2021 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants