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.

Sign up for GitHub

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

Jump to bottom

ci: avoid command duplication in CI and use set -x to visualize what is ran #54

Merged
merged 1 commit into from
Oct 28, 2024
Merged

ci: avoid command duplication in CI and use set -x to visualize what is ran #54

merged 1 commit into from
Oct 28, 2024

Conversation

yarikoptic
Copy link
Member

@yarikoptic yarikoptic commented Oct 18, 2024 •
edited
Loading

Just treating my duplication allergy

TODO:

  • check that it works as desired (note: set -x better be on the same line to be more "descriptive" instead of just set -x)
    image

@yarikoptic yarikoptic requested a review from mih as a code owner October 18, 2024 16:21
@codeclimate CodeClimate
Copy link

codeclimate bot commented Oct 18, 2024

Code Climate has analyzed commit 3cf9946 and detected 0 issues on this pull request.

View more on Code Climate.

mih
mih approved these changes Oct 28, 2024
View reviewed changes
Copy link
Member

@mih mih left a comment

Choose a reason for hiding this comment

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

Thanks!

@mih mih merged commit 0d2b470 into main Oct 28, 2024
4 checks passed
@mih mih deleted the use-set-x branch October 28, 2024 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mih mih mih approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yarikoptic @mih