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
Add some DAP endpoints to spec file #143
Conversation
92cd761
to
c842285
Compare
87df6c0
to
5cf2653
Compare
56aca51
to
b22afaf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor remaining comments to fix up before merging
fc6708b
to
ebb9c3d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - can we get #160 in first, though?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you need a follow up to update the release process to do OSS only, let's plan to do that separate from merging this PR
Also added python integration tests for the three added endpoints. /health /remote_health /info
Also removed contract test and linting from Jenkins pipeline. They are always run in github actions so having them in Jenkins is redundant. We only need the integration tests to run in Jenkins for test artifacts/coverage reporting.
Run the local script in place of manually performing the linting steps in a GitHub action. This should make sure linting checks locally match CI
What does this PR do?
Adds some of the DAP endpoints to the OpenAPI spec and adds integration tests for the new endpoints as well as infrastructure to run them.
In the name of minimizing the amount of time spent on this one PR I have excluded two things which issues should be opened to add later:
configuration/{account}/seed/follower
endpointBefore the Jenkins CI tests will run properly this PR on dap-intro needs to be merged in.Follow up issues:
What ticket does this PR close?
Resolves #144
Checklists
Change log
Test coverage
Documentation
README
s) were updated in this PR, and/or there is a follow-on issue to update docs, or