Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

216 security #217

Merged
merged 4 commits into from
Jun 4, 2018
Merged

216 security #217

merged 4 commits into from
Jun 4, 2018

Conversation

Retro64
Copy link
Collaborator

@Retro64 Retro64 commented Jun 4, 2018

close #216

@Retro64 Retro64 requested review from ISO50 and bzums June 4, 2018 07:21
@coveralls
Copy link

coveralls commented Jun 4, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 67baecf on 216-security into 9662a05 on master.

ISO50
ISO50 reviewed Jun 4, 2018
View reviewed changes
integration-test/oauth-tooling/getAccessToken.spec.ts Outdated
@@ -12,6 +12,7 @@ import {
RefreshGrantConfig,
OAuthConfig
} from '../../src';
// import { equal } from 'assert';
Copy link
Collaborator

Choose a reason for hiding this comment

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

comment

@ISO50
Copy link
Collaborator

ISO50 commented Jun 4, 2018

👍

1 similar comment
@Retro64
Copy link
Collaborator Author

Retro64 commented Jun 4, 2018

👍

@Retro64 Retro64 merged commit 6a4ad30 into master Jun 4, 2018
@Retro64 Retro64 deleted the 216-security branch June 4, 2018 12:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ISO50 ISO50 ISO50 left review comments

@bzums bzums Awaiting requested review from bzums

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cleanup dependencies
3 participants
@Retro64 @coveralls @ISO50