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

Fix deprecation of set-output #79

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Conversation

consideRatio
Copy link
Member

@consideRatio consideRatio commented Nov 14, 2022

@consideRatio consideRatio force-pushed the pr/set-output branch 2 times, most recently from e199882 to f471d63 Compare November 14, 2022 12:40
@consideRatio consideRatio changed the title ci: fix deprecation of set-output in github workflows Fix deprecation of set-output Nov 14, 2022
action.yml Outdated Show resolved Hide resolved
echo "::set-output name=helm-version::$(helm version --short | sed 's/\([^+]*\).*/\1/')"
echo "kubeconfig=$HOME/.kube/config" >> $GITHUB_OUTPUT
echo "k3s-version=$(k3s --version | grep --max-count=1 'k3s' | sed 's/.*\(v[0-9][^ ]*\).*/\1/')" >> $GITHUB_OUTPUT
echo "k8s-version=$(k3s --version | grep --max-count=1 'k3s' | sed 's/.*\(v[0-9][^+]*\).*/\1/')" >> $GITHUB_OUTPUT
Copy link
Member Author

Choose a reason for hiding this comment

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

This emitted two lines, so a bugfix was introduced here to get only ome.

Copy link
Member Author

Choose a reason for hiding this comment

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

K8s version is assumed from k3s version, so grep of k3s x2 is intended.

@consideRatio
Copy link
Member Author

consideRatio commented Nov 15, 2022

I added a changelog entry for v3.0.6 into this like you did in #73 @manics. Agree to go for a patch release on merge?

EDIT

I extracted the changelog part, wanting to see two more PRs merged, still reasoning its a patch release to follow though.

@manics manics merged commit 01f5c3e into jupyterhub:main Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants