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

Sprint7 91 staging merge #130

Merged
9 commits merged into from Oct 1, 2020
Merged

Sprint7 91 staging merge #130

9 commits merged into from Oct 1, 2020

Conversation

ghost
Copy link

@ghost ghost commented Oct 1, 2020

No description provided.

kaladay and others added 9 commits September 29, 2020 15:03
…issing adocs.

The productsStatsApi.adoc and the remoteProjectStatsAPI.adoc are missing.
The RepoTest.java has everything that IntegrationTest.java needs and reproduces.
There may be extra data in the class tree, but this does not cause any observable problems.
…pproved.

Once this is reviewed and approved, this branch and commit should be rebuilt and replaced to point to master branch.
Relocate the cached controller API to the top as they are what is most likely desired.

Remove remoteProjectAPI.adoc and RemoteProjectControllerIntegrationTest.java..

Make sure the index.adoc has all of the appropriate files.

Fix incorrect path, `/status` clearly should instead be `/remote-project-manager`.

Remove the FIXME, this appears to now be working with the removal of RemoteProjectControllerIntegrationTest.java and/or remoteProjectAPI.adoc.

Filenames need to use `API` and not `Api`.
@ghost ghost requested review from kaladay and rladdusaw October 1, 2020 13:19
@ghost ghost merged commit e2aaa32 into sprint7-staging Oct 1, 2020
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 95.756% when pulling d6cc9d4 on sprint7-91-staging-merge into caca898 on sprint7-staging.

@ghost ghost deleted the sprint7-91-staging-merge branch November 30, 2020 16:44
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants