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

Private PR: Review changes of the final to commits to cove is_str() changes in datapath.py #37

Open
wants to merge 6 commits into
base: feature/py3
Choose a base branch
from

Conversation

bernhardkaindl
Copy link
Collaborator

For review of the final two commits to:

  • add testing with code coverage for datapath.py

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
…is_str

Unused imports of is_long() are removed as well.

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
…nings

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
…o is_str()

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
…her.attach

Signed-off-by: Bernhard Kaindl <bernhard.kaindl@cloud.com>
@bernhardkaindl bernhardkaindl force-pushed the storage-api-cleanup-string-checks-with-test-coverage branch from c23f314 to 290a653 Compare July 18, 2024 16:22
Copy link

sonarcloud bot commented Jul 18, 2024

@coveralls
Copy link

Pull Request Test Coverage Report for Build 9995121301

Details

  • 109 of 109 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.2%) to 75.174%

Totals Coverage Status
Change from base Build 9994789573: -2.2%
Covered Lines: 3985
Relevant Lines: 5301

💛 - Coveralls

@xenserver-next xenserver-next deleted a comment from github-actions bot Jul 18, 2024
@xenserver-next xenserver-next deleted a comment from github-actions bot Jul 18, 2024
@psafont
Copy link

psafont commented Jul 19, 2024

What's producing the sonarcloud message?

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