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

✨ : github module import #198

Merged
merged 13 commits into from
Jan 15, 2020
Merged

✨ : github module import #198

merged 13 commits into from
Jan 15, 2020

Conversation

juwit
Copy link
Member

@juwit juwit commented Jan 3, 2020

No description provided.

@juwit juwit force-pushed the github_module_import branch from 35030fc to 09b2c49 Compare January 3, 2020 15:21
@juwit juwit requested a review from cdubuisson January 3, 2020 15:31
@juwit juwit added ✨ feature:modules module list & module edition 👌 ready for review The pull request is ready for review ✨ enhancement New feature or request labels Jan 3, 2020
@juwit juwit force-pushed the github_module_import branch 6 times, most recently from 6f0fff7 to 61591fe Compare January 8, 2020 12:14
juwit and others added 13 commits January 9, 2020 08:37
When passing parameters to the RestTemplate exchange method, spring
url-encodes the parameter. This behaviour is not desirable for
GithubRawContent, as the project ID contains a '/'
Github serves raw content in base 64 with line feeds for large files. We have to remove them before decoding the content.
`Mockito.verifyZeroInteractions` is deprecated since 3.x.x. Using
`verifyNoInteraction` instead
to remote the need of mockito inline
@juwit juwit force-pushed the github_module_import branch from 61591fe to 86eecd3 Compare January 9, 2020 07:37
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

90.5% 90.5% Coverage
0.0% 0.0% Duplication

@cdubuisson cdubuisson self-assigned this Jan 13, 2020
@cdubuisson cdubuisson merged commit 131b0e1 into master Jan 15, 2020
@cdubuisson cdubuisson deleted the github_module_import branch January 15, 2020 13:21
@juwit juwit mentioned this pull request Feb 3, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
✨ enhancement New feature or request ✨ feature:modules module list & module edition 👌 ready for review The pull request is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants