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

policy search and variables enpoints & doc updates #515

Merged

Conversation

cianfoley-nearform
Copy link
Contributor

Rather than rebase these changes against master as per PR #512, I am re PRing here...

@ShogunPanda, you might take a scan of the hapi 17 routes and tests to make sure I did that OK

@coveralls
Copy link

coveralls commented Apr 5, 2018

Coverage Status

Coverage decreased (-0.02%) to 93.114% when pulling ee6fa82 on policy-search-variables-post-hapi-17-merge into eb10745 on master.

@cianfoley-nearform cianfoley-nearform mentioned this pull request Apr 6, 2018
@ShogunPanda ShogunPanda requested review from ShogunPanda and paolochiodi and removed request for paolochiodi April 6, 2018 10:51
Copy link
Contributor

@ShogunPanda ShogunPanda left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@dberesford dberesford left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @cianfoley-nearform

@cianfoley-nearform cianfoley-nearform merged commit 5fb9c8c into master Apr 6, 2018
@cianfoley-nearform cianfoley-nearform deleted the policy-search-variables-post-hapi-17-merge branch May 23, 2018 08:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants