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

Update README.md #49

Merged
merged 2 commits into from
Nov 19, 2024
Merged

Update README.md #49

merged 2 commits into from
Nov 19, 2024

Conversation

adamant-pwn
Copy link
Contributor

It seems the project is not uploaded to PyPI and should be installed manually

It seems the project is not uploaded to PyPI and should be installed manually
@adamant-pwn
Copy link
Contributor Author

Actually it seems that PyPI deployment was added in #36, so I'm not sure I fully understand why the package is not there.

@akahles
Copy link
Member

akahles commented Oct 29, 2024

Package is now on PyPI. Maybe we no longer need this?

@adamant-pwn
Copy link
Contributor Author

Returned PyPI mention and amended this into just adding a separate section. I think it's still useful, as the later method makes it easier to quickly fix something in the archiver locally if it suddenly throws an error. I think it's still be better to add.

@akahles
Copy link
Member

akahles commented Nov 19, 2024

Thanks - Looks good

@akahles akahles merged commit f71630f into ratschlab:main Nov 19, 2024
2 checks passed
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