Skip to content

Update CONTRIBUTING.md to have clear IDEA Formatting setup instructions#35448

Merged
gsmet merged 1 commit intoquarkusio:mainfrom
chrischiedo:patch-2/contributing-doc-update
Aug 22, 2023
Merged

Update CONTRIBUTING.md to have clear IDEA Formatting setup instructions#35448
gsmet merged 1 commit intoquarkusio:mainfrom
chrischiedo:patch-2/contributing-doc-update

Conversation

@chrischiedo
Copy link
Copy Markdown
Contributor

Short summary
The current instructions for setting up formatting using Eclipse Code Formatter in IDEA are a bit confusing.

Details
I just went through setting up my InjelliJ IDEA IDE for working on the Quarkus project, and I realized that the instructions for setting up formatting using Eclipse Code Formatter were a bit inconsistent with the actual steps that I took to accomplish the task.

Version of IDE used: IntelliJ IDEA 2023.2 (Community Edition).

Reason for PR
Update the instructions for setting up formatting standard in IDEA to reflect the correct steps (that I actually followed).

Caveat: Not sure if there is a variation in the instructions between Ultimate Edition and the Community Edition of the IDE.

@chrischiedo chrischiedo marked this pull request as ready for review August 21, 2023 18:37
@chrischiedo chrischiedo force-pushed the patch-2/contributing-doc-update branch from 6f81a12 to e21cb69 Compare August 22, 2023 05:45
Copy link
Copy Markdown
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Thanks so much for taking the time to adjust these instructions to the new versions.

@gsmet gsmet merged commit 838137e into quarkusio:main Aug 22, 2023
@quarkus-bot quarkus-bot bot added this to the 3.4 - main milestone Aug 22, 2023
@chrischiedo chrischiedo deleted the patch-2/contributing-doc-update branch August 22, 2023 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants