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 ci #646

Merged
merged 3 commits into from
Sep 24, 2024
Merged

fix ci #646

merged 3 commits into from
Sep 24, 2024

Conversation

JackCrumpLeys
Copy link
Contributor

@JackCrumpLeys JackCrumpLeys commented Sep 15, 2024

https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/:

With the introduction of v4, we will be deprecating v1 and v2 of actions/upload-artifact, actions/download-artifact, and related npm packages on June 30, 2024. We strongly encourage customers to update their workflows to begin using v4 of the artifact actions.

EDIT: I have also updated deps to fix issues with CI

https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/:
> With the introduction of v4, we will be deprecating v1 and v2 of
actions/upload-artifact, actions/download-artifact, and related npm
packages on June 30, 2024. We strongly encourage customers to update
their workflows to begin using v4 of the artifact actions.
@dyc3
Copy link
Collaborator

dyc3 commented Sep 16, 2024

I don't have permissions to override the requirements and merge, the unused deps job needs to pass.

@JackCrumpLeys
Copy link
Contributor Author

@dyc3 Looks like its passing!

@dyc3 dyc3 enabled auto-merge (squash) September 24, 2024 01:55
@dyc3 dyc3 merged commit ce37618 into valence-rs:main Sep 24, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants