Skip to content

Re-enable compiler warnings in :test:framework#75449

Merged
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:40366-compiler-warnings-test-framework
Jul 21, 2021
Merged

Re-enable compiler warnings in :test:framework#75449
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:40366-compiler-warnings-test-framework

Conversation

@pugnascotia
Copy link
Copy Markdown
Contributor

Part of #40366.

@pugnascotia pugnascotia requested a review from breskeby July 19, 2021 10:37
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Jul 19, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@sethmlarson sethmlarson added the Team:Clients Meta label for clients team label Jul 19, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/clients-team (Team:Clients)

Copy link
Copy Markdown
Contributor

@breskeby breskeby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. TIL about @SafeVarargs

@pugnascotia pugnascotia added auto-backport Automatically create backport pull requests when merged and removed auto-backport Automatically create backport pull requests when merged labels Jul 21, 2021
@pugnascotia pugnascotia merged commit cd44467 into elastic:master Jul 21, 2021
@pugnascotia pugnascotia deleted the 40366-compiler-warnings-test-framework branch July 21, 2021 12:47
pugnascotia added a commit that referenced this pull request Jul 21, 2021
@pugnascotia
Copy link
Copy Markdown
Contributor Author

Backported in 400029a.

ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request Jul 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure >non-issue Team:Clients Meta label for clients team Team:Delivery Meta label for Delivery team v7.15.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants