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.

Sign up for GitHub

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

Jump to bottom

declare const headers #117

Merged
merged 3 commits into from
May 11, 2022
Merged

declare const headers #117

merged 3 commits into from
May 11, 2022

Conversation

lperlaki
Copy link
Contributor

@lperlaki lperlaki commented May 6, 2022

now that hyperium/http#499 is merged and released, the headers can be declared as const instead of static Lazy

@lperlaki
Copy link
Contributor Author

@Nugine could you rerun the ci, I think addressed the clippy errors

@Nugine
Copy link
Contributor

Nugine commented May 11, 2022

Thanks for your improvement!

@Nugine Nugine merged commit cf67039 into datenlord:master May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nugine Nugine Nugine approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lperlaki @Nugine