BetaHuhn / repo-file-sync-action

πŸ”„ GitHub Action to keep files like Action workflows or entire directories in sync between multiple repositories.
https://github.com/marketplace/actions/repo-file-sync-action
MIT License
290 stars 84 forks source link

Actions fail in v1.17.18 #219

Closed svg153 closed 2 years ago

svg153 commented 2 years ago

🐞 Describe the bug

After this merge, our workflows failed. Look at the screenshots.

πŸ“š To Reproduce

πŸ’‘ Expected behavior

Works like version v1.17.17

πŸ–ΌοΈ Screenshots

Action trace screenshot ![failed](https://user-images.githubusercontent.com/9192031/172156859-696a106d-5fba-4231-8f08-e8309a6c595f.png)
As seen in the commit screenshot. ![commit](https://user-images.githubusercontent.com/9192031/172156861-574a20ff-7f09-42b7-9a87-22b4b8d8b5c1.png)

βš™οΈ Environment

πŸ“‹ Additional context

etremblay commented 2 years ago

We have the same issue

BetaHuhn commented 2 years ago

Thanks for reporting this @svg153!

Issue should be fixed in v1.17.19. Please let me know if the error is resolved on your end.

The issue was caused by an automated dependency upgrade, specifically node-readfiles v0.3.0. To mitigate such an issue from disrupting the use of this action in the future I have disabled all automatic dependency updates for now. I will keep it disabled until we have near 100% test coverage of this action (currently we have none).

I think this is a good opportunity to think about adding tests to this project, if anyone has recommendations on how to do this, please let me know! Also happy to accept contributions if anyone wants to work in this themselves. I created discussion #220 to discuss this further.

Thanks!

svg153 commented 2 years ago

It works again with the master version πŸ‘ ❀️

Thanks :)

PD: For us, this issue would be closed