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

base/message_builder: add missing OptBuilder methods #61

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

vavrusa
Copy link
Contributor

@vavrusa vavrusa commented Jun 2, 2020

This allows OptBuilder to set DO flag and change EDNS version.

@vavrusa
Copy link
Contributor Author

vavrusa commented Jun 3, 2020

The build failed with clippy errors unrelated to this PR, do you want me to fix those here?

@partim
Copy link
Member

partim commented Jun 4, 2020

I know it is a tiny thing, but could you move the ring update to a separate PR?

I’ll fix the Clippy complaint without a PR directly in master.

@vavrusa
Copy link
Contributor Author

vavrusa commented Jun 4, 2020

No problem! I removed the commit and will open a separate PR after this.

This allows OptBuilder to set DO flag and change EDNS version.
@vavrusa
Copy link
Contributor Author

vavrusa commented Jun 15, 2020

Should I bump the release version or are you going to do it separately?

@partim
Copy link
Member

partim commented Jun 16, 2020

Oh, darn. I must have missed the notification for your update. Merging now. Sorry.

@partim partim merged commit 7e13535 into NLnetLabs:master Jun 16, 2020
@partim
Copy link
Member

partim commented Jun 16, 2020

I’ll do the ring update.

@partim
Copy link
Member

partim commented Jun 16, 2020

Actually, your commit for that disappeared – which version did you want for ring?

@vavrusa
Copy link
Contributor Author

vavrusa commented Jun 16, 2020

No problem! I repushed it in a new PR.

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