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

Update Project Dependencies #1

Merged
merged 3 commits into from
Mar 8, 2018
Merged

Update Project Dependencies #1

merged 3 commits into from
Mar 8, 2018

Conversation

zachary-kuhn
Copy link
Contributor

Description

The dependencies of the project are out of date. In particular, the Sirupsen/logrus dependency which made the decision to use all lowercase. In the process of updating all dependencies, I discovered the yahoo/athenz library also changed its API, so I made a fix there.

What Changed?

  • Update Glide dependencies
  • Change Sirupsen/logrus to sirupsen/logrus
  • Change zts.EntityName to zts.EntityList to match the updated Yahoo Athenz API
  • Change Procuder typo to Producer

How Was This Tested?

  • Successfully ran make
  • Ran example command line commands:
$ ./bin/pulsar-client --conf example/default.ini --command produce --topic "persistent://sample/standalone/ns1/my-topic" --messages "Hello Pulsar"
$ ./bin/pulsar-client --conf example/default.ini --command consume --topic "persistent://sample/standalone/ns1/my-topic" --subscriptionName sub

This was tested on Mac OS X with Go 1.10 darwin/amd64.

@t2y t2y merged commit b953ed4 into t2y:master Mar 8, 2018
@t2y
Copy link
Owner

t2y commented Mar 8, 2018

Thank you for updating!

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