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

fix CI #142

Merged
merged 1 commit into from
Dec 15, 2023
Merged

fix CI #142

merged 1 commit into from
Dec 15, 2023

Conversation

amtoine
Copy link
Owner

@amtoine amtoine commented Dec 15, 2023

related to failed jobs on the nightly branch...

description

this PR adds a little step to show the target branch and the current branch and fixes the logic to set the Nushell and Nupm revisions.

@amtoine amtoine added ci Something related to the Continuous Integration fix A fix for a bug labels Dec 15, 2023
@amtoine
set CI variables correctly when on a branch
bc938ff 
this commit separates the logic to set the versions of Nushell and Nupm
to a standalone job to help with the syntax 👀
@amtoine
Copy link
Owner Author

amtoine commented Dec 15, 2023

i really hope this will work 😆 😱

@amtoine amtoine closed this Dec 15, 2023
@amtoine amtoine reopened this Dec 15, 2023
@amtoine amtoine merged commit 2d0e2c0 into main Dec 15, 2023
11 checks passed
@amtoine amtoine deleted the fix-ci branch December 15, 2023 16:55
@amtoine amtoine mentioned this pull request Dec 15, 2023
Merged
amtoine added a commit that referenced this pull request Dec 15, 2023
@amtoine
thanks Bash (missing quotes in boolean tests for undefined variables) (…
dcb5140 
…#143)

follow-up to
- #142
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci Something related to the Continuous Integration fix A fix for a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@amtoine