Skip to content

fix: use -r with jq and add workflow for artifacts #5016

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
Merged
merged 1 commit into from
Mar 22, 2022
Merged

fix: use -r with jq and add workflow for artifacts #5016

merged 1 commit into from
Mar 22, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Mar 22, 2022

Uses raw output when using jq so branch name isn't in quotes and adds workflow for artifact download.

Fixes N/A

@jsjoeio jsjoeio self-assigned this Mar 22, 2022
@jsjoeio jsjoeio requested a review from a team March 22, 2022 23:45
@github-actions
Copy link

✨ code-server docs for PR #5016 is ready! It will be updated on every commit.

@jsjoeio jsjoeio merged commit ba1ddbd into main Mar 22, 2022
@jsjoeio jsjoeio deleted the jsjoeio/fix-workflow branch March 22, 2022 23:46
@jsjoeio jsjoeio added this to the March 2022 milestone Mar 22, 2022
@jsjoeio jsjoeio temporarily deployed to npm March 22, 2022 23:50 Inactive
TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
@jsjoeio @TinLe
fix: use -r with jq and add workflow for artifacts (coder#5016)
a0c4521
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@code-asher code-asher code-asher approved these changes

Assignees

@jsjoeio jsjoeio

Labels

None yet

Projects

None yet

Milestone

4.2.0

Development

Successfully merging this pull request may close these issues.

2 participants

@jsjoeio @code-asher