Skip to content

Conversation

@koppor
Copy link
Member

@koppor koppor commented Sep 13, 2025

After our reformatting is through, this is the howto

Should be finished after #13883 got merged.

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [/] I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@koppor koppor force-pushed the add-codestyle-howto branch from 3b6f14f to 86f6219 Compare September 13, 2025 17:27
@koppor koppor marked this pull request as ready for review September 13, 2025 20:20
@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Sep 13, 2025
@trag-bot
Copy link

trag-bot bot commented Sep 13, 2025

@trag-bot didn't find any issues in the code! ✅✨

@koppor koppor enabled auto-merge September 13, 2025 20:38
@koppor koppor added this pull request to the merge queue Sep 13, 2025
Merged via the queue into main with commit 766c89e Sep 13, 2025
41 checks passed
@koppor koppor deleted the add-codestyle-howto branch September 13, 2025 20:56
Siedlerchr added a commit that referenced this pull request Sep 14, 2025
* upstream/main:
  Add hint on auto formatting (#13886)
  Add ADR-0050 on code formatter (#13895)
  Reformat codebase (more carefully) (#13885)
  New Crowdin updates (#13892)
  Fix record wrapping (#13889)
  Fix JavaDoc (#13888)
  Improve IntelliJ settings (#13887)
  Manual code reformattings (#13883)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project: jabcon status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants