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

Update to work with latest Rocket RC #109

Merged
merged 3 commits into from
Jun 6, 2022
Merged

Update to work with latest Rocket RC #109

merged 3 commits into from
Jun 6, 2022

Conversation

ELD
Copy link
Collaborator

@ELD ELD commented Jun 6, 2022

The following changes were made:

  • Satisfying the #[must_use] annotation on the Rocket struct where
    necessary (mostly in test code)
  • Adding the http crate for the HeaderName type
    • Rocket no longer exports all of hyper or http
      indescriminately, so we have to be more mindful of types from
      either crate that we require. Thus, this commit adds the http
      crate to satisfy the requirement.
  • Updated tests to properly return a result from a launched Rocket

ELD and others added 3 commits June 5, 2022 20:42
The following changes were made:
- Satisfying the `#[must_use]` annotation on the `Rocket` struct where
  necessary (mostly in test code)
- Adding the `http` crate for the `HeaderName` type
    - Rocket no longer exports all of `hyper` or `http`
      indescriminately, so we have to be more mindful of types from
      either crate that we require. Thus, this commit adds the `http`
      crate to satisfy the requirement.
- Updated tests to properly return a result from a launched `Rocket`
@ELD ELD requested a review from lawliet89 June 6, 2022 02:55
@ELD ELD assigned ELD and lawliet89 Jun 6, 2022
@ELD ELD enabled auto-merge (squash) June 6, 2022 02:58
@ELD ELD merged commit 54fae07 into master Jun 6, 2022
@ELD ELD deleted the rocket-rc2-updates branch June 6, 2022 03:03
@lawliet89
Copy link
Owner

Thanks for this @ELD. LGTM :)

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