-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 version pinning of thrift, ugoriji/go, and etcd #2457
Conversation
For incompatibility reasons at one point, we were forced to pining the version of thrift, ugoriji/go, and etcd to very specific versions (some are not versioned commits) to get around the build issues. It looks like those incompatibility issues are gone so we could remove the pinning of thrift, ugoriji/go, and etcd. Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Thank you for your contribution. I've just checked the OWNERS files to find a suitable reviewer. This search was successful and I've asked varyoo (via If you have questions or suggestions for this bot, please file an issue against the miekg/dreck repository. The bot understands the commands that are listed here. |
Codecov Report
@@ Coverage Diff @@
## master #2457 +/- ##
=========================================
+ Coverage 56.37% 56.4% +0.02%
=========================================
Files 208 208
Lines 10490 10490
=========================================
+ Hits 5914 5917 +3
+ Misses 4124 4122 -2
+ Partials 452 451 -1
Continue to review full report at Codecov.
|
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM by stp-ip
Remove version pinning of thrift, ugoriji/go, and etcd (coredns#2457)
* Remove version pinning of thrift, ugoriji/go, and etcd For incompatibility reasons at one point, we were forced to pining the version of thrift, ugoriji/go, and etcd to very specific versions (some are not versioned commits) to get around the build issues. It looks like those incompatibility issues are gone so we could remove the pinning of thrift, ugoriji/go, and etcd. Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Update Gopkg.lock Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Update vendor library Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Remove version pinning of thrift, ugoriji/go, and etcd For incompatibility reasons at one point, we were forced to pining the version of thrift, ugoriji/go, and etcd to very specific versions (some are not versioned commits) to get around the build issues. It looks like those incompatibility issues are gone so we could remove the pinning of thrift, ugoriji/go, and etcd. Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Update Gopkg.lock Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Update vendor library Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
1. Why is this pull request needed and what does it do?
For incompatibility reasons at one point, we were forced
to pining the version of thrift, ugoriji/go, and etcd
to very specific versions (some are not versioned commits)
to get around the build issues.
It looks like those incompatibility issues are gone so
we could remove the pinning of thrift, ugoriji/go, and etcd.
2. Which issues (if any) are related?
3. Which documentation changes (if any) need to be made?
Signed-off-by: Yong Tang yong.tang.github@outlook.com