Skip to content

Configure required IDEA plugins#84674

Merged
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:required-idea-plugins
Mar 7, 2022
Merged

Configure required IDEA plugins#84674
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:required-idea-plugins

Conversation

@pugnascotia
Copy link
Copy Markdown
Contributor

Closes #53812.

The CheckStyle-IDEA and EcliseCodeFormatter IDEA plugins are to all
intents and purposes required when using IntelliJ to work on
Elasticsearch, so add configuration that marks then as required.

We already version configuration snippets for both plugins.

Closes elastic#53812.

The CheckStyle-IDEA and EcliseCodeFormatter IDEA plugins are to all
intents and purposes required when using IntelliJ to work on
Elasticsearch, so add configuration that marks then as required.
@pugnascotia pugnascotia added >non-issue :Delivery/Tooling Developer tooliing and automation v8.2.0 labels Mar 4, 2022
@pugnascotia pugnascotia requested a review from mark-vieira March 4, 2022 16:43
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Mar 4, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

👍

@mark-vieira
Copy link
Copy Markdown
Contributor

I mistakenly commited to master my local changes here. I've reverted so it's fine to merge/backport this now.

@pugnascotia pugnascotia merged commit 223c2e4 into elastic:master Mar 7, 2022
@pugnascotia pugnascotia deleted the required-idea-plugins branch March 7, 2022 10:35
pugnascotia added a commit that referenced this pull request Mar 7, 2022
Closes #53812.

The CheckStyle-IDEA and EcliseCodeFormatter IDEA plugins are to all
intents and purposes required when using IntelliJ to work on
Elasticsearch, so add configuration that marks then as required.
pugnascotia added a commit that referenced this pull request Mar 7, 2022
Closes #53812.

The CheckStyle-IDEA and EcliseCodeFormatter IDEA plugins are to all
intents and purposes required when using IntelliJ to work on
Elasticsearch, so add configuration that marks then as required.
@mark-vieira
Copy link
Copy Markdown
Contributor

FYI, we need to be backporting any build/IDE stuff to 7.17 as well as that branch is going to be around for awhile.

pugnascotia added a commit that referenced this pull request Mar 7, 2022
Closes #53812.

The CheckStyle-IDEA and EcliseCodeFormatter IDEA plugins are to all
intents and purposes required when using IntelliJ to work on
Elasticsearch, so add configuration that marks then as required.
@pugnascotia
Copy link
Copy Markdown
Contributor Author

Done 👍

@mark-vieira
Copy link
Copy Markdown
Contributor

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Tooling Developer tooliing and automation >non-issue Team:Delivery Meta label for Delivery team v8.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatically configure formatting for IntelliJ

3 participants