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

Rename new method on Packet types to new_checked #254

Closed
wants to merge 1 commit into from

Conversation

whitequark
Copy link
Contributor

Fixes #195.

r? @dlrobertson, did I miss anything?

@whitequark
Copy link
Contributor Author

@m-labs-homu try

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
@m-labs-homu
Copy link

⌛ Trying commit 4be0c8b with merge 077a0dc...

@m-labs-homu
Copy link

💔 Test failed - status-travis

@whitequark
Copy link
Contributor Author

@m-labs-homu retry

@m-labs-homu
Copy link

⌛ Trying commit 4be0c8b with merge 87ec5fa...

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
@m-labs-homu
Copy link

💔 Test failed - status-travis

@whitequark
Copy link
Contributor Author

@m-labs-homu retry

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
@m-labs-homu
Copy link

⌛ Trying commit 4be0c8b with merge c13ceef...

@m-labs-homu
Copy link

☀️ Test successful - status-travis
State: approved= try=True

@dlrobertson
Copy link
Collaborator

Looks good! I like how explicit new_checked and new_unchecked are.

@whitequark
Copy link
Contributor Author

@m-labs-homu r=dlrobertson

@m-labs-homu
Copy link

📌 Commit 4be0c8b has been approved by dlrobertson

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
Fixes #195.

Closes: #254
Approved by: dlrobertson
@m-labs-homu
Copy link

⌛ Testing commit 4be0c8b with merge 2a5be8d...

@whitequark
Copy link
Contributor Author

@m-labs-homu r-
Forgot one.

@whitequark
Copy link
Contributor Author

@m-labs-homu r=dlrobertson

@m-labs-homu
Copy link

📌 Commit 23acdf5 has been approved by dlrobertson

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
Fixes #195.

Closes: #254
Approved by: dlrobertson
@m-labs-homu
Copy link

⌛ Testing commit 23acdf5 with merge 72070c9...

@m-labs-homu
Copy link

💔 Test failed - status-travis

@whitequark
Copy link
Contributor Author

@m-labs-homu retry

@whitequark
Copy link
Contributor Author

@m-labs-homu r=dlrobertson

@m-labs-homu
Copy link

📌 Commit 9cb7e18 has been approved by dlrobertson

@m-labs-homu
Copy link

⌛ Testing commit 9cb7e18 with merge 4d7b43f...

m-labs-homu pushed a commit that referenced this pull request Jul 11, 2018
Fixes #195.

Closes: #254
Approved by: dlrobertson
@m-labs-homu
Copy link

💔 Test failed - status-travis

@whitequark
Copy link
Contributor Author

@m-labs-homu r=dlrobertson

@m-labs-homu
Copy link

📌 Commit 680f675 has been approved by dlrobertson

@m-labs-homu
Copy link

⌛ Testing commit 680f675 with merge 4a253fe...

@m-labs-homu
Copy link

☀️ Test successful - status-travis
Approved by: dlrobertson
Pushing 4a253fe to master...

@whitequark whitequark deleted the new_checked branch August 17, 2020 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants