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

excludes command parameter from token #968

Merged
merged 2 commits into from Sep 25, 2019

Conversation

@shamsimam
Copy link
Contributor

commented Sep 24, 2019

Changes proposed in this PR

  • excludes command parameter from token

Why are we making these changes?

Avoid command in token tests unless needed.

@shamsimam shamsimam self-assigned this Sep 24, 2019
@shamsimam shamsimam force-pushed the version-string-token-test branch from fb323a7 to 30855a2 Sep 24, 2019
@shamsimam shamsimam force-pushed the version-string-token-test branch from feea064 to a399609 Sep 25, 2019
@shamsimam shamsimam requested a review from dposada Sep 25, 2019
@dposada

This comment has been minimized.

Copy link
Member

commented Sep 25, 2019

Will merge when green

@dposada dposada merged commit c1c1eac into master Sep 25, 2019
2 checks passed
2 checks passed
Mergeable Mergeable Run has been Completed!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dposada dposada deleted the version-string-token-test branch Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.