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

Uses Python built-in modules to retrieve Python and auth0-python version number #125

Merged
merged 3 commits into from Oct 8, 2018
Merged

Conversation

ghost
Copy link

@ghost ghost commented Oct 2, 2018

No description provided.

Copy link
Contributor

@lbalmaceda lbalmaceda left a comment

Choose a reason for hiding this comment

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

Thanks. The class was missing tests to cover the telemetry header. Would you mind adding at least one that asserts that the Auth0-Client header is present and with the correct value? Should be located on the auth0/v3/test/management/test_rest.py file

@ghost
Copy link
Author

ghost commented Oct 6, 2018

Okay, I'll add the test. Please wait.

@ghost
Copy link
Author

ghost commented Oct 6, 2018

@lbalmaceda please review the added test. Thanks!

@lbalmaceda
Copy link
Contributor

Looks good. Thanks for the changes. I've found that the Auth calls are missing Telemetry too and created an issue to track this. Since it's similar to what you've done, feel free to take it 👌 #136

@lbalmaceda lbalmaceda merged commit c4b03a9 into auth0:master Oct 8, 2018
@lbalmaceda lbalmaceda added this to the v3-Next milestone Oct 8, 2018
@ghost
Copy link
Author

ghost commented Oct 9, 2018

@lbalmaceda Thank you for the merge. I think I'll pass on the opportunity, but thank you!

@ghost ghost deleted the nicer-version-number branch October 9, 2018 00:01
@lbalmaceda lbalmaceda modified the milestones: v3-Next, 3.4.0 Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants