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

fix warnings for Crystal 0.35.1 #38

Merged
merged 1 commit into from
Oct 26, 2020
Merged

Conversation

sam0x17
Copy link
Contributor

@sam0x17 sam0x17 commented Jun 27, 2020

Fixes a few warnings from Crystal 0.35.1 (io contract change, gzip)

@Sija
Copy link
Contributor

Sija commented Sep 6, 2020

@schovi Ping! It's needed for the Crystal 0.35.1+ versions.

@straight-shoota straight-shoota merged commit fb07911 into schovi:master Oct 26, 2020
@straight-shoota
Copy link
Collaborator

I realized I have write access to this repo, too. I just can't approve (and thus merge) my own PRs (#35).

Thanks @sam0x17 and everyone else involved: @Sija, @carlhoerberg, @repomaa. I chose to merge this PR of the lot because it seemed to be the most complete and don't add additional features. For future reference it would be helpful to check if there are already PRs pending for the same purpose.

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.

3 participants