Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test PR to enable CLA #1

Closed
wants to merge 1 commit into from
Closed

Conversation

SergeyKanzhelev
Copy link
Member

No description provided.

@SergeyKanzhelev SergeyKanzhelev deleted the SergeyKanzhelev-patch-1 branch November 6, 2019 05:21
trask pushed a commit that referenced this pull request May 4, 2020
* Normalize sql statements to elide literal numbers and strings.

* Missed one SlickTest sql normalization.

* Fix muzzle order for helper classes.

* Change name of feature flag

* Upgrade to latest spotless version in an attempt #1 to make the circleci build work.

* Attempt 2 to make circleci build happy - exclude build/generated/** from spotless.

* Attempt 3 to get circleci build working, adding *.java to the exclude line.

* Change exclude of generated files to include of just the src/ directory.
I confirmed that this properly failed the build if I remove a license header from a src/ directory.
iNikem referenced this pull request in iNikem/opentelemetry-java-instrumentation May 5, 2020
…telemetry#366)

* Normalize sql statements to elide literal numbers and strings.

* Missed one SlickTest sql normalization.

* Fix muzzle order for helper classes.

* Change name of feature flag

* Upgrade to latest spotless version in an attempt #1 to make the circleci build work.

* Attempt 2 to make circleci build happy - exclude build/generated/** from spotless.

* Attempt 3 to get circleci build working, adding *.java to the exclude line.

* Change exclude of generated files to include of just the src/ directory.
I confirmed that this properly failed the build if I remove a license header from a src/ directory.
anuraaga added a commit that referenced this pull request Jun 28, 2020
#584)

* Don't set IP address for net.peer.name and set name for failed DNS lookup

* Private

* Don't require IPv6 to run unit test

* PR suggestion (#1)

* PR suggestion

* Make test pass on Windows

* final

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@ghost ghost mentioned this pull request Oct 14, 2020
iNikem pushed a commit that referenced this pull request Nov 3, 2020
* apache camel 2.20.x instrumentation

* removed instrumentation advice from helper classes

* code review #1

* code review #2 / experimental server name update

* code review changes / new tests, improved direct, new license header

* rebase changes

* code review changes

* code review changes, REST tests added

* changes after rebase to newest master

* code review changes

* code review

* code review

* rebase to master
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.

1 participant