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

Feature/downloader #41

Merged
merged 9 commits into from
Mar 7, 2023
Merged

Feature/downloader #41

merged 9 commits into from
Mar 7, 2023

Conversation

bramstoeller
Copy link
Contributor

No description provided.

@bramstoeller bramstoeller linked an issue Oct 25, 2022 that may be closed by this pull request
@bramstoeller bramstoeller disabled auto-merge October 26, 2022 09:50
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 1, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

TonyXiang8787
TonyXiang8787 previously approved these changes Nov 3, 2022
@bramstoeller
Copy link
Contributor Author

We need more unit tests in order to merge this branch. @TonyXiang8787, we didn't really plan time for this story in this sprint. Would you like me (or someone else) to write unit tests for it? Or do we want to decrease the test coverage limit?

@bramstoeller bramstoeller added the needs-unit-tests More unit tests are needed label Nov 3, 2022
@bramstoeller bramstoeller removed the needs-unit-tests More unit tests are needed label Mar 1, 2023
@bramstoeller bramstoeller force-pushed the feature/downloader branch 3 times, most recently from dc73d0e to 5f8f32a Compare March 2, 2023 14:11
@bramstoeller bramstoeller removed a link to an issue Mar 2, 2023
@petersalemink95
Copy link
Member

Looks good, had some minor comments.

petersalemink95
petersalemink95 previously approved these changes Mar 6, 2023
petersalemink95
petersalemink95 previously approved these changes Mar 7, 2023
@bramstoeller bramstoeller merged commit d4bb730 into main Mar 7, 2023
@bramstoeller bramstoeller deleted the feature/downloader branch March 7, 2023 11:41
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.

3 participants