search

test-room-7 / action-update-file

Update (i.e. commit and push) files on GitHub
MIT License
40 stars 10 forks source link

binary files are committed incorrectly #5

Closed dukess closed 4 years ago

dukess commented 4 years ago

Hello! I tried using the script and I liked the result, except that when copying binary files (gzipped javascript), they break. Is it possible to fix this? Workflow file: https://github.com/dukess/bsb_lan/blob/master/.github/workflows/localization.yaml

alexesprit commented 4 years ago

Hi! Thanks for reporting the issue.

Not sure why it happens; perhaps binary files should be updated using a different way. I'll investigate this issue.

dukess commented 4 years ago

If it helps: I tried add .gitattributes file to root directory with *.js.gz binary but nothing changes (or i did something wrong)

alexesprit commented 4 years ago

Looks like I figured out why the actions corrupts binary files.

I made an attempt to fix this in create-blob branch, though I can't test changes now - GitHub Actions are down. I will try to test changes later, and we'll see if this fix works or not.

dukess commented 4 years ago

Thank you! Now binary files committing correctly.

dukess commented 4 years ago

Вопрос уже философский, на будущее: насколько велики шансы, что этот скрипт будет в будщем удалён, имеет ли смысл утащить его к себе?

alexesprit commented 4 years ago

Не понял, про какой скрипт идёт речь?

dukess commented 4 years ago

Собственно, про этот: test-room-7/action-update-file Пожалуй, неудачно назвал скриптом.

alexesprit commented 4 years ago

Ааа, ты про экшен. Нет, я не буду его удалять, зачем? Я сам им пользуюсь в своих проектах. Сегодня ещё протестирую и залью обновление бинарных файлов в основную ветку.

dukess commented 4 years ago

Ок. Ещё раз спасибо!