Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Fix/options #40

Merged
merged 2 commits into from
Jan 10, 2019
Merged

Fix/options #40

merged 2 commits into from
Jan 10, 2019

Conversation

nachoesmite
Copy link
Contributor

Allow injection of properties in options field and inject at root level of request-promise reqOpts

alerodm
alerodm approved these changes Jan 10, 2019
View reviewed changes
Copy link

@alerodm alerodm left a comment

Choose a reason for hiding this comment

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

LGTM

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 52.427% when pulling f750cc1 on fix/options into 064a3cc on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 52.427% when pulling f750cc1 on fix/options into 064a3cc on master.

@nachoesmite nachoesmite merged commit 0d23475 into master Jan 10, 2019
@nachoesmite nachoesmite deleted the fix/options branch January 10, 2019 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alerodm alerodm alerodm approved these changes

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@nachoesmite @coveralls @alerodm