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
Add a function to list all files in a archive #23
Conversation
Pull Request Test Coverage Report for Build 149870013
💛 - Coveralls |
4e02fb1
to
2b20921
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Asakiz thanks for looking into it. Please take a look at the comments I made below.
Another thing we need to look at is because the minimal Rust version will need to be moved to 1.40.0 due to the doctest
feature use. Please add it as another commit.
2b20921
to
85f66ec
Compare
2f6c5ec
to
7ac6ca5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for looking into it. The changes look good.
The only change I did was to invert the changes so both commits can pass the CI checks.
7ac6ca5
to
b0f79a6
Compare
Signed-off-by: asakiz <asakizin@gmail.com>
Fixes: #22 Signed-off-by: asakiz <asakizin@gmail.com> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
b0f79a6
to
e51c023
Compare
Prior merging it I made few minor adjustments:
|
As requested, I tried the master branch, and it works nicely. Thank you for your efforts! |
@Aloso thanks for testing it. We made the 0.6.0 release. |
Signed-off-by: asakiz asakizin@gmail.com