Skip to content

Conversation

@ShaunSHamilton
Copy link
Member

Even if I end up not using this repo for the Exam Environment stuff, I saw the state of the Action, and could not leave it.

  • s3-sync-action is archived
  • uploading all the files all the time is unnecessary

We might want to consider running with --dry-run initially.

Copy link
Member

@raisedadead raisedadead left a comment

Choose a reason for hiding this comment

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

Here is a fitting review 😈😤:

chore is NOT a valid conventional commit because this is a clear refactor.

Also this actions should not need to comment 7-8 lines about the rclone docs. Devs can RTFM.

Also thanks a lot.

@ShaunSHamilton ShaunSHamilton changed the title chore(.github): use rclone for efficient upload refactor(.github): use rclone (refactor) for efficient (refactor) upload Dec 11, 2025
@raisedadead raisedadead merged commit 63ca9ee into main Dec 11, 2025
2 checks passed
@raisedadead raisedadead deleted the chore_rclone-gha branch December 11, 2025 16:05
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