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

Remove deprecated validation funcs #333

Merged
merged 1 commit into from
Feb 20, 2020
Merged

Conversation

appilon
Copy link
Contributor

@appilon appilon commented Feb 20, 2020

No description provided.

prune unused from internal/addrs
@appilon appilon requested review from paultyng, paddycarver and a team February 20, 2020 19:10
@@ -1,14 +0,0 @@
package addrs
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this file getting removed in this PR?

Copy link
Contributor Author

@appilon appilon Feb 20, 2020

Choose a reason for hiding this comment

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

I just wanted to sneak it in... its dead code, I know not the best habit, it somehow was missed in my formal cleanup PR, staticcheck seems to be a bit inconsistent.

@appilon appilon changed the title remove deprecated validation funcs Remove deprecated validation funcs Feb 20, 2020
@appilon appilon merged commit 5185da1 into version2 Feb 20, 2020
@appilon appilon deleted the remove-deprecated-in-v1 branch February 20, 2020 20:51
@appilon appilon added this to the v2.0.0 milestone Feb 21, 2020
@appilon appilon added the breaking-change Implementation which breaks compatibility or other promises label Feb 21, 2020
@ghost
Copy link

ghost commented Mar 22, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Mar 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking-change Implementation which breaks compatibility or other promises
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants