Skip to content

Update rust.yml#345

Closed
Daulox92 wants to merge 1 commit intonexus-xyz:mainfrom
Daulox92:patch-3
Closed

Update rust.yml#345
Daulox92 wants to merge 1 commit intonexus-xyz:mainfrom
Daulox92:patch-3

Conversation

@Daulox92
Copy link
Contributor

Update actions/checkout from v2 to v4 in Spartan workflow

Changes Made:
spartan/.github/workflows/rust.yml:

  • uses: actions/checkout@v2
  • uses: actions/checkout@v4

Why:

  • Improves performance and security
  • Adds better support for modern Git features
  • Aligns with other workflows that already use v4
  • Follows GitHub Actions best practices

All other workflows in the repository are already using checkout@v4, making this change consistent across the codebase.

sjudson pushed a commit that referenced this pull request Feb 5, 2025
* impl

* indexing bug

* sam + duc comments

* fix
sjudson pushed a commit that referenced this pull request Feb 12, 2025
* impl

* indexing bug

* sam + duc comments

* fix
@slumber
Copy link
Contributor

slumber commented Feb 22, 2025

Note that this action is part of forked spartan repo and is never run.

Our changes to the fork are outlined in the readme and we want to keep it commit-by-commit reviewable.

@slumber slumber closed this Feb 22, 2025
sjudson pushed a commit that referenced this pull request Mar 11, 2025
* impl

* indexing bug

* sam + duc comments

* fix
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.

2 participants