-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Description
Re-opening unresolved Issue because @ericsciple bizarrely closes it here #2365 (comment)
Are you saying that files are actually missing from the checkout itself? Or is rsync detecting fewer changes than expected?
rsync is detecting fewer changes as a direct result of this bump so it is not resolved
If it's the latter, that would be unrelated to checkout - rsync compares files in two directories, and checkout just puts files on disk.
They are absolutely related, this action is literally creating those files on disk, including the actual timestamps for them for rsync to read
Again, I must reiterate, the only change before bug is this action got bumped, once reverted to 6.0.1 (workflow file remaining unchanged) rsync works perfectly again. It is not my job to hunt for what specific part of the breaking changes you or @TingluoHuang made affected checkout behavior such that rsync can no longer detect timestamps properly (probably git history related)
Closing for now, but please reopen if you have more details indicating the checkout itself produced different files.
Not sure why I would have to provide "more details" for an unfixed bug to remain open, but above was additional info, below is original helpful info:
Here using 6.0.2 https://github.com/adamlui/js-utils/actions/runs/21301896443/job/61321359067
![]()
.. it fails to catch a bunch of changed files.
Here using reverted 6 .0.1 https://github.com/adamlui/js-utils/actions/runs/21302606217/job/61323620602
![]()
... it gets them all. Workflow file is https://github.com/adamlui/js-utils/blob/0b53ea8be64cf39c9f1005bc04d6f23bd3a66ab7/.github/workflows/sync-minify.js-changes-to-repos.yml

