Skip to content

[BEAM-1385] Add license header to website Jenkins job descriptions#1912

Merged
asfgit merged 1 commit intoapache:masterfrom
kennknowles:groovy-RAT
Feb 3, 2017
Merged

[BEAM-1385] Add license header to website Jenkins job descriptions#1912
asfgit merged 1 commit intoapache:masterfrom
kennknowles:groovy-RAT

Conversation

@kennknowles
Copy link
Member

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 838ef04 on kennknowles:groovy-RAT into ** on apache:master**.

@asfbot
Copy link

asfbot commented Feb 3, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/7052/
--none--

@kennknowles
Copy link
Member Author

Because this unbreaks the build, I am going to merge before review.

@kennknowles
Copy link
Member Author

R: @davorbonaci

@asfgit asfgit merged commit 838ef04 into apache:master Feb 3, 2017
asfgit pushed a commit that referenced this pull request Feb 3, 2017
@kennknowles
Copy link
Member Author

(or any committer, since we are now switching which side of the Earth is in daylight :-)

@davorbonaci
Copy link
Member

LGTM.

Thanks @kennknowles; much appreciated. FYI @jasonkuster.

@jasonkuster
Copy link
Contributor

@peihe created #1911 to fix too, looks like this got in before his.

@kennknowles kennknowles deleted the groovy-RAT branch March 31, 2017 23:57
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.

6 participants