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

Introduce smallrye-common-io #36

Merged
merged 1 commit into from
Jul 15, 2020
Merged

Introduce smallrye-common-io #36

merged 1 commit into from
Jul 15, 2020

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Jul 15, 2020

@radcortez
Copy link
Member

Does it make sense to add some tests?

@gastaldi
Copy link
Contributor Author

I thought about that too, but I wasn't certain what would that do since all methods just do new JarFile

@radcortez
Copy link
Member

No worries, let's keep it this way. I guess you need a release soon? :)

@gastaldi
Copy link
Contributor Author

That would be nice, thanks 👍

@radcortez radcortez merged commit 0b011ac into smallrye:master Jul 15, 2020
@gastaldi gastaldi deleted the io branch July 15, 2020 16:59
dmlloyd added a commit that referenced this pull request Jul 16, 2020
Due to the `smallrye-common-io` proposed changes (#36/#37)
@radcortez radcortez added this to the 1.1.0 milestone Jul 20, 2020
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