Skip to content

Conversation

@umbynos
Copy link
Contributor

@umbynos umbynos commented Apr 19, 2021

Updated binary name (from FirmwareUpdater to FirmwareUploader) in the CI / Taskfiles / .gitignore.
I think it's a bit confusing to use both FirmwareUploader and FirmwareUpdater in the same GitHub repo though.. I've renamed also the repo handle. The only thing missing is to rename the repo name in settings.

@umbynos umbynos self-assigned this Apr 19, 2021
@umbynos
Copy link
Contributor Author

umbynos commented Apr 21, 2021

CI checks are failing because I changed also links to the repo.

@umbynos umbynos requested review from a team and cmaglie April 21, 2021 14:09
Copy link
Contributor

@silvanocerza silvanocerza left a comment

Choose a reason for hiding this comment

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

Just two small changes to do.

Also shouldn't we change extras/package_index.json.template too? The tool there is still called fwupdater.

@umbynos
Copy link
Contributor Author

umbynos commented Apr 21, 2021

I did not rename these two files and not updated the readme accordingly because they are used in the js lib.

Regarding the template:
the name fwupdater will be changed in the future but it will eventually cause breaking changes.

@umbynos umbynos merged commit cef9dfc into main Apr 21, 2021
@cmaglie cmaglie deleted the umbynos/rename_firmware_uploader branch April 21, 2021 15:03
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