Skip to content

TRIVIAL: extend list of classes exported from SDK init#132

Merged
1 commit merged into
gooddata:masterfrom
jaceksan:working
Aug 12, 2022
Merged

TRIVIAL: extend list of classes exported from SDK init#132
1 commit merged into
gooddata:masterfrom
jaceksan:working

Conversation

@jaceksan

Copy link
Copy Markdown
Contributor

handy when processing declarative documents and want to type arguments of functions properly

handy when processing declarative documents and want to type arguments of functions properly
@ghost

ghost commented Aug 12, 2022

Copy link
Copy Markdown

Build succeeded (check pipeline).

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #132 (585af3e) into master (42929f5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #132   +/-   ##
=======================================
  Coverage   91.75%   91.75%           
=======================================
  Files          71       71           
  Lines        4426     4427    +1     
=======================================
+ Hits         4061     4062    +1     
  Misses        365      365           
Impacted Files Coverage Δ
gooddata-sdk/gooddata_sdk/__init__.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@hkad98 hkad98 self-requested a review August 12, 2022 15:58
@hkad98 hkad98 added the merge label Aug 12, 2022
@yenkins

yenkins commented Aug 12, 2022

Copy link
Copy Markdown

Sonar scan result

More detail, see in https://sonarqube-gate.intgdc.com/dashboard?id=gooddata-python-sdk-gate-PR132

To scan for vulnerabilities in dependencies and run unit tests (to get coverage report in sonar) please comment your PR with 'extended check sonar'.

@ghost ghost removed the merge label Aug 12, 2022
@ghost ghost merged commit 845b899 into gooddata:master Aug 12, 2022
@ghost

ghost commented Aug 12, 2022

Copy link
Copy Markdown

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.

4 participants