Skip to content

Don't apply the spotless plugin to non-applicable projects #80679

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

Merged

Conversation

mark-vieira
Copy link
Contributor

This is an attempt at resolving, or at least reducing the prevalence of #77837. A more robust solution is probably required, but this should help reduce this problem since we now apply spotless to about 90 fewer subprojects.

@mark-vieira mark-vieira added >non-issue :Delivery/Build Build or test infrastructure v8.0.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport-and-merge v7.16.1 labels Nov 11, 2021
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Nov 11, 2021
@elasticmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine merged commit 025dbdc into elastic:master Nov 11, 2021
@mark-vieira mark-vieira deleted the attempted_spotless_fix branch November 11, 2021 23:17
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Nov 11, 2021
…0679)

This is an attempt at resolving, or at least reducing the prevalence of
elastic#77837. A more robust solution is probably required, but this should
help reduce this problem since we now apply spotless to about 90 fewer
subprojects.
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Nov 11, 2021
…0679)

This is an attempt at resolving, or at least reducing the prevalence of
elastic#77837. A more robust solution is probably required, but this should
help reduce this problem since we now apply spotless to about 90 fewer
subprojects.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.0
7.16

elasticsearchmachine pushed a commit that referenced this pull request Nov 12, 2021
…80681)

This is an attempt at resolving, or at least reducing the prevalence of
#77837. A more robust solution is probably required, but this should
help reduce this problem since we now apply spotless to about 90 fewer
subprojects.
elasticsearchmachine pushed a commit that referenced this pull request Nov 12, 2021
…80680)

This is an attempt at resolving, or at least reducing the prevalence of
#77837. A more robust solution is probably required, but this should
help reduce this problem since we now apply spotless to about 90 fewer
subprojects.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v7.16.0 v8.0.0-rc1 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants