Skip to content

Have StatusCode::as_str return static reference#380

Open
upsuper wants to merge 1 commit intohyperium:masterfrom
upsuper-forks:as-str-static
Open

Have StatusCode::as_str return static reference#380
upsuper wants to merge 1 commit intohyperium:masterfrom
upsuper-forks:as-str-static

Conversation

@upsuper
Copy link
Copy Markdown
Contributor

@upsuper upsuper commented Jan 3, 2020

No description provided.

@upsuper upsuper changed the title Have Status::as_str return static reference Have StatusCode::as_str return static reference Jan 3, 2020
Copy link
Copy Markdown
Contributor

@dekellum dekellum left a comment

Choose a reason for hiding this comment

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

I'm trying to fix CI in #381. This change looks good to me.

@upsuper
Copy link
Copy Markdown
Contributor Author

upsuper commented Jan 25, 2020

@dekellum Hey! I saw that #381 is closed (not sure what happened there). What about this PR now? Should I rebase or?

@dekellum
Copy link
Copy Markdown
Contributor

CI was restored in a different way (#387), so you might try to merge with master to get this PR passing.

@upsuper
Copy link
Copy Markdown
Contributor Author

upsuper commented Jan 25, 2020

Thanks!

@upsuper
Copy link
Copy Markdown
Contributor Author

upsuper commented Jan 27, 2020

So... the CI is still broken.

I'd suggest you mark nightly with allow_failures, especially given that you have deny(warnings) which is very easy to be broken.

@upsuper
Copy link
Copy Markdown
Contributor Author

upsuper commented Jan 29, 2020

@dekellum could you have another look? it seems that I can't re-request review, and I have no idea why.

@dekellum
Copy link
Copy Markdown
Contributor

Your rebased change continues to look good to me!

But I have no commit rights here, nor do I endorse, much less understand, its "governance". All resent commits are by @seanmonstar and @carllerche was previously active. You might try requesting a review from them?

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.

2 participants