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

Update checkstyle config for newest plugin version #88919

Merged
merged 2 commits into from
Aug 22, 2022

Conversation

mark-vieira
Copy link
Contributor

@mark-vieira mark-vieira commented Jul 28, 2022

The IntelliJ checkstyle plugin has changed the format of it's configuration file in a recent update. I believe this is the cause of some occasional errors trying to read the configuration.

@mark-vieira mark-vieira added >non-issue :Delivery/Tooling Developer tooliing and automation labels Jul 28, 2022
@mark-vieira mark-vieira requested a review from pugnascotia July 28, 2022 21:19
@elasticsearchmachine elasticsearchmachine added v8.5.0 Team:Delivery Meta label for Delivery team labels Jul 28, 2022
@elasticsearchmachine
Copy link
Collaborator

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

<option value="$PROJECT_DIR$/build-conventions/build/libs/build-conventions.jar" />
</option>
<option name="activeLocationIds">
<option value="dfb90780-ced3-4152-a4df-3cc9be08e3d8" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Any idea what this is?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's just an auto-generated UUID referencing the id in ConfigurationLocation element below.

@mark-vieira
Copy link
Contributor Author

@elasticmachine update branch

@mark-vieira mark-vieira merged commit 6b962ef into elastic:main Aug 22, 2022
@mark-vieira mark-vieira deleted the checkstyle_plugin_config branch August 22, 2022 17:01
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Aug 24, 2022
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Aug 24, 2022
elasticsearchmachine pushed a commit that referenced this pull request Aug 24, 2022
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 v7.17.7 v8.4.1 v8.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants