Skip to content

[Hotfix] fix rabbitmp spotless check - #11059

Merged
kennknowles merged 1 commit into
apache:masterfrom
vectorijk:rabbitmp-spotless
Mar 5, 2020
Merged

[Hotfix] fix rabbitmp spotless check#11059
kennknowles merged 1 commit into
apache:masterfrom
vectorijk:rabbitmp-spotless

Conversation

@vectorijk

Copy link
Copy Markdown
Contributor

per https://builds.apache.org/job/beam_PreCommit_Spotless_Commit/7888/console

Post-Commit Tests Status (on master branch)

LangSDKApexDataflowFlinkGearpumpSamzaSpark
GoBuild Status------Build Status------Build Status
JavaBuild StatusBuild StatusBuild Status
Build Status
Build Status
Build Status
Build Status
Build StatusBuild StatusBuild Status
Build Status
Build Status
PythonBuild Status
Build Status
Build Status
Build Status
---Build Status
Build Status
Build Status
Build Status
Build Status
------Build Status
XLang---------Build Status------Build Status

Pre-Commit Tests Status (on master branch)

---JavaPythonGoWebsite
Non-portableBuild StatusBuild Status
Build Status
Build StatusBuild Status
Portable---Build Status------

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@vectorijk

Copy link
Copy Markdown
ContributorAuthor

R: @alexvanboxel
cc: @Hk-tang

@vectorijk

Copy link
Copy Markdown
ContributorAuthor

R: @amaliujia this causes spotless jenkins test failure on master (in #11058https://builds.apache.org/job/beam_PreCommit_Spotless_Commit/7888/)

@kennknowles

Copy link
Copy Markdown
Member

LGTM

@kennknowles

Copy link
Copy Markdown
Member

Spotless passed. Merging.

@kennknowles
kennknowles merged commit 90e4f60 into apache:masterMar 5, 2020
@alexvanboxel

Copy link
Copy Markdown
Contributor

yes, my rookie mistake. should have seen that the PR didn't have the spotless ran. Sorry

@amaliujia

Copy link
Copy Markdown
Contributor

Thanks @vectorijk!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@vectorijk@kennknowles@alexvanboxel@amaliujia