Skip to content

FEATURE: MSF-22857 Sub module gooddata_brick_base to import in bricks#1849

Merged
1 commit merged intogooddata:masterfrom
hung-nguyen-hoang:MSF-22857
Apr 19, 2022
Merged

FEATURE: MSF-22857 Sub module gooddata_brick_base to import in bricks#1849
1 commit merged intogooddata:masterfrom
hung-nguyen-hoang:MSF-22857

Conversation

@hung-nguyen-hoang
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Apr 4, 2022

@hung-nguyen-hoang
Copy link
Contributor Author

ok to test

@ghost
Copy link

ghost commented Apr 4, 2022

@hung-nguyen-hoang
Copy link
Contributor Author

ok to test

@ghost
Copy link

ghost commented Apr 5, 2022

@ghost
Copy link

ghost commented Apr 7, 2022

@ghost
Copy link

ghost commented Apr 7, 2022

@hung-nguyen-hoang hung-nguyen-hoang force-pushed the MSF-22857 branch 2 times, most recently from 52a61d3 to 64f89c3 Compare April 12, 2022 02:54
@ghost
Copy link

ghost commented Apr 12, 2022

Build in pipeline check aborted.

@ghost
Copy link

ghost commented Apr 12, 2022


module GoodData
module Bricks
class PipelineBase
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to BasePipeline for consitence, change file name also please.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done update code. Thank you.

@ghost
Copy link

ghost commented Apr 15, 2022

@sangtm sangtm added the merge label Apr 19, 2022
@yenkins
Copy link

yenkins commented Apr 19, 2022

Sonar scan result

More detail, see in https://sonarqube-gate.intgdc.com/dashboard?id=gooddata-ruby-gate-PR1849

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 Apr 19, 2022
@ghost ghost merged commit 07e9596 into gooddata:master Apr 19, 2022
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.

3 participants