Skip to content

Conversation

brightly-salty
Copy link
Contributor

Fixes #300

@yoshuawuyts yoshuawuyts marked this pull request as draft December 24, 2020 13:53
Copy link
Member

@yoshuawuyts yoshuawuyts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

@yoshuawuyts yoshuawuyts added the semver-major This change requires a semver major change label Dec 24, 2020
@brightly-salty brightly-salty marked this pull request as ready for review January 11, 2021 16:27
@joshtriplett
Copy link
Member

This needs a rebase to fix the merge conflicts (and to separate out the formatting changes which should now be redundant), but otherwise this looks good to me.

@joshtriplett
Copy link
Member

@brightly-salty That merge doesn't seem to have addressed the conflicts. Also, please don't back-merge main into a PR branch.

Could you rebase this, please?

@brightly-salty
Copy link
Contributor Author

@joshtriplett Okay, I've rebased it but I'm afraid I might have resolved the conflicts incorrectly. Let me know if something is wrong.

@Fishrock123 Fishrock123 dismissed their stale review March 3, 2021 20:35

addressed

OneOfOne added a commit to OneOfOne/http-types-rs that referenced this pull request Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-major This change requires a semver major change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move trailers submodule to transfer
4 participants