Skip to content

Conversation

@Cosss7
Copy link

@Cosss7 Cosss7 commented Jun 18, 2020

No description provided.

@apache-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-111/1/console in case of problems.

@apache-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 16m 50s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 shellcheck 0m 2s There were no new shellcheck issues.
+1 💚 shelldocs 0m 0s There were no new shelldocs issues.
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 10s The patch does not generate ASF License warnings.
17m 17s
Subsystem Report/Notes
Docker ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-111/1/artifact/out/Dockerfile
GITHUB PR #111
Optional Tests dupname asflicense shellcheck shelldocs
uname Linux 7ad9a0f95bcc 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality precommit/src/main/shell/personality/yetus.sh
git revision master / b218355
Max. process+thread count 41 (vs. ulimit of 2000)
modules C: precommit U: precommit
Console output https://builds.apache.org/job/yetus-github-multibranch/job/PR-111/1/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.7.0
Powered by Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@aw-was-here
Copy link
Contributor

Hmm. Looks like my Gitlab setup has fallen into disrepair. Let me get it fixed here so I can test this out. :)

@aw-was-here
Copy link
Contributor

👍 Please squash this into a single commit and then we can get it merged since it looks like if I squash it, it gives credit to me instead of you. :(

Thanks!

@aw-was-here aw-was-here self-requested a review June 20, 2020 22:45
@Cosss7
Copy link
Author

Cosss7 commented Jun 21, 2020

Done.

@aw-was-here
Copy link
Contributor

Merged via smart-apply-patch.

Thanks!

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