Skip to content

[HOTFIX] Exclude org/apache/zeppelin/scio/avro/* once at pom.xml#1706

Closed
tae-jun wants to merge 1 commit intoapache:masterfrom
tae-jun:patch-2
Closed

[HOTFIX] Exclude org/apache/zeppelin/scio/avro/* once at pom.xml#1706
tae-jun wants to merge 1 commit intoapache:masterfrom
tae-jun:patch-2

Conversation

@tae-jun
Copy link
Contributor

@tae-jun tae-jun commented Nov 30, 2016

What is this PR for?

Previously on #1676, I excluded path twice. It's my mistake! @khalidhuseynov pointed that out (Thanks!).

What type of PR is it?

Hot Fix

How should this be tested?

Build with command mvn clean package install -DskipTests -DskipRat

Questions:

  • Does the licenses files need update? NO
  • Is there breaking changes for older versions? NO
  • Does this needs documentation? NO

@zjffdu
Copy link
Contributor

zjffdu commented Nov 30, 2016

LGTM

@khalidhuseynov
Copy link
Member

LGTM!

@tae-jun
Copy link
Contributor Author

tae-jun commented Nov 30, 2016

@zjffdu @khalidhuseynov Thanks! 😄

@astroshim
Copy link
Contributor

LGTM.

@Leemoonsoo
Copy link
Member

LGTM and merge to master.

@tae-jun
Copy link
Contributor Author

tae-jun commented Nov 30, 2016

@astroshim @Leemoonsoo Thanks for review!

@asfgit asfgit closed this in 249cf6c Nov 30, 2016
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.

5 participants