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

refactor: remove wget dependency #69

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

Blackvote
Copy link
Contributor

@Blackvote Blackvote commented Sep 13, 2024

Remove wget.
Add http.get(gaia.url) - io.copy(gaia.path)
also add os.arch check (only arm64 or amd64)
close #51

@hanchon
Copy link
Owner

hanchon commented Sep 16, 2024

Awesome! I'll review the code later today, thanks for the PR!

@hanchon
Copy link
Owner

hanchon commented Sep 16, 2024

I just fixed the linter issues, now it's ready to be merged. Thanks

@hanchon hanchon changed the title Feat remove wget dependency refactor: remove wget dependency Sep 16, 2024
@hanchon hanchon merged commit 83e78f6 into hanchon:main Sep 16, 2024
2 checks passed
@Blackvote
Copy link
Contributor Author

That's my first pr. Thanks! =)

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.

reactor: remove wget dependency
2 participants