Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Auto formatting configuration #304

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

andreaTP
Copy link
Member

@andreaTP andreaTP commented Aug 9, 2022

Opening this PR as Draft to receive comments/suggestions regarding the proposed format.
I already attempted a little to make it follow the current formatting, but I think some of the original maintainers of the project should chip in for this decision.

Try the formatting with:

mvn clean install -f ide-config/pom.xml
mvn formatter:format

Revert the formatting with:

git checkout HEAD **/*.java

If someone wants to provide an alternative file I'm completely open to it 🙂

@andreaTP andreaTP requested review from jsenko and EricWittmann August 9, 2022 18:59
@andreaTP
Copy link
Member Author

I dropped in this configuration:
https://github.com/Apicurio/apicurio-configs/blob/main/eclipse/formatter.xml

but seems to be breaking a little more than the original proposal.

In the longer term, we can publish the configuration directly from https://github.com/Apicurio/apicurio-configs instead of having to spin out a separate maven sub-project in each repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant