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

✨ Fix version catalog writing #724

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MaxMichel2
Copy link
Contributor

Hello/bonjour,

What?

Resolves #705

  • Changed the symbol used for available versions in the version catalog
  • Changed the TOML sections order for the version catalog

Why?

Fixes the issues with version alignment in the version catalog which was dependent on the font used in the IDE. The symbol used is now font-agnostic and is always the same size.

This PR also fixes the somewhat annoying issue where the sections of the version catalog were re-ordered. The order now follows what most people will expect and also what is described in the Gradle documentation (See here).

How?

  • Replaced the symbol with the ^ character
  • Set the TOML sections order to the following:
    • Root
    • [versions]
    • [libraries]
    • [bundles]
    • [plugins]

Testing?

Published to mavenLocal and tested in a local project using refreshVersions

On some fonts, the arrow symbol was only 1 character wide instead of 2. Using a classic character negates this issue
@MaxMichel2 MaxMichel2 mentioned this pull request Aug 28, 2024
1 task
@MaxMichel2 MaxMichel2 changed the title Bugfix/version catalog writing ✨ Fix version catalog writing Aug 28, 2024
@MaxMichel2
Copy link
Contributor Author

FYI, the check has failed due to some signing configuration that doesn't seem to work (This hasn't been modified so I suspect the issue lies on your side since I have no way of handling this)

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.

Version alignement issue
1 participant