Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename FirmwareUpdater to FirmwareUploader #21

Merged
merged 4 commits into from Apr 21, 2021

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 cmaglie and a team 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.

None yet

2 participants