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

Stop using the Save Actions plugin (broken in IntelliJ 2023.1) and use native format on save #862

Merged
merged 3 commits into from
Apr 6, 2023

Conversation

CRogers
Copy link
Contributor

@CRogers CRogers commented Apr 6, 2023

Before this PR

We use the Save Actions IntelliJ plugin but it is broken in IntelliJ 2023.1 and is not maintained and will not have a new release.

After this PR

==COMMIT_MSG==
Stop using the Save Actions plugin (broken in IntelliJ 2023.1) and use native format on save (requires IntelliJ >=2021.2)
==COMMIT_MSG==

In 2021.2, IntelliJ introduced a new built in reformat on save feature. We configure this instead rather than using a custom plugin.

Possible downsides?

The suggesting of Save Actions lives in baseline unfortunately

@changelog-app
Copy link

changelog-app bot commented Apr 6, 2023

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Stop using the Save Actions plugin (broken in IntelliJ 2023.1) and use native format on save (requires IntelliJ >=2021.2)

Check the box to generate changelog(s)

  • Generate changelog entry

@bulldozer-bot bulldozer-bot bot merged commit 7697268 into develop Apr 6, 2023
@bulldozer-bot bulldozer-bot bot deleted the callumr/fix-save-actions-2023.1 branch April 6, 2023 15:45
@svc-autorelease
Copy link
Collaborator

Released 2.29.0

@ash211
Copy link
Contributor

ash211 commented Apr 6, 2023

This addresses palantir/gradle-baseline#2545

@CRogers CRogers mentioned this pull request Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants