-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 types to HTTP::StaticFileHandler
#13778
Add types to HTTP::StaticFileHandler
#13778
Conversation
This adds types to StaticFuleHandler initialize and move the assignment to the arg assignemnt
Hm, this is technically a breaking change because things like So maybe we should add an unrestricted overload just to be safe? # :ditto:
@[Deprecated]
def self.new(public_dir : String, fallthrough = true, directory_listing = true)
new(public_dir, fallthrough: fallthrough, listing: listing)
end I don't know... maybe that's too much caution. I cannot see an actual justified use case for a non-Bool type parameter. |
81d358f
to
73f16b1
Compare
Co-authored-by: Sijawusz Pur Rahnama <sija@sija.pl>
Why add the breaking change label again? It should no longer be due to the new overload. |
It adds a depreciation and deprecations are marked as breaking changes. |
Don't we now have a separate deprecation tag |
Oops right. Guess I was out on vacation for too long 😏 |
HTTP::StaticFileHandler
Co-authored-by: Sijawusz Pur Rahnama <sija@sija.pl> Co-authored-by: Jack Thorne <jack@myrenee.io>
This adds types to StaticFuleHandler initialize and move the assignment to the arg assignemnt