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(actions): update datahub actions default version #348

Merged
merged 4 commits into from
Oct 20, 2023

Conversation

DSchmidtDev
Copy link
Contributor

Updated the default version according to actual releases https://github.com/acryldata/datahub-actions/releases

@anshbansal
Copy link
Contributor

The chart version also need to be updated otherwise this will not work.

@DSchmidtDev
Copy link
Contributor Author

Ah thought it can go in the next release but updated the chart version now :)

@DSchmidtDev
Copy link
Contributor Author

@anshbansal @iprentic someone can review this?

@DSchmidtDev
Copy link
Contributor Author

@shirshanka maybe you can approve this? :)

@github-actions
Copy link

This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io.

@github-actions github-actions bot added the stale label Sep 16, 2023
@pedro93
Copy link
Contributor

pedro93 commented Sep 19, 2023

@DSchmidtDev can you please resolve the merge conflict? I will approve after.

@DSchmidtDev
Copy link
Contributor Author

DSchmidtDev commented Sep 19, 2023

@pedro93 resolved the merge conflict

@github-actions github-actions bot removed the stale label Sep 20, 2023
@github-actions
Copy link

This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io.

@github-actions github-actions bot added the stale label Oct 20, 2023
@OneCyrus
Copy link

stumbled across the same outdated actions image. any chance this can get merged? @pedro93

@pedro93
Copy link
Contributor

pedro93 commented Oct 20, 2023

Sorry about the delay here folks. Approved the pr.

@pedro93 pedro93 merged commit a20eac2 into acryldata:master Oct 20, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants