Skip to content

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

Merged
1 commit merged into
gooddata: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 into
gooddata:masterfrom
hung-nguyen-hoang:MSF-22857

Conversation

@hung-nguyen-hoang

Copy link
Copy Markdown
Contributor

No description provided.

@ghost

ghost commented Apr 4, 2022

Copy link
Copy Markdown

@hung-nguyen-hoang

Copy link
Copy Markdown
Contributor Author

ok to test

@ghost

ghost commented Apr 4, 2022

Copy link
Copy Markdown

@hung-nguyen-hoang

Copy link
Copy Markdown
Contributor Author

ok to test

@ghost

ghost commented Apr 5, 2022

Copy link
Copy Markdown

@ghost

ghost commented Apr 7, 2022

Copy link
Copy Markdown

@ghost

ghost commented Apr 7, 2022

Copy link
Copy Markdown

@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

ghost commented Apr 12, 2022

Copy link
Copy Markdown

Build in pipeline check aborted.

@ghost

ghost commented Apr 12, 2022

Copy link
Copy Markdown

Comment thread lib/gooddata/bricks/pipeline_base.rb Outdated

module GoodData
module Bricks
class PipelineBase

Copy link
Copy Markdown

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
Copy Markdown
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

ghost commented Apr 15, 2022

Copy link
Copy Markdown

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

yenkins commented Apr 19, 2022

Copy link
Copy Markdown

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