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

Wrong formatting for "install" command #54

Open
b-nathan opened this issue Jan 22, 2025 · 2 comments
Open

Wrong formatting for "install" command #54

b-nathan opened this issue Jan 22, 2025 · 2 comments

Comments

@b-nathan
Copy link

gersemi --version
gersemi 0.17.0
lark 1.2.2
colorama (missing)
Python 3.9.2 (default, Dec  1 2024, 12:12:57) 
[GCC 10.2.1 20210110]

This is how gersemi formatts the install section:

install(
    TARGETS
        ${NAME}
        DESTINATION
        bin
)

And this is how I think it should be formatted, according to the CMake Documenation:

install(
    TARGETS
        ${NAME}
    DESTINATION
        bin
)

What rules follows gersemi for formatting and why does it not format as expected?

This is my .gersemirc:

# This is the config file for the CMakeLists.txt file formatter "gersemi"
# yaml-language-server: $schema=https://raw.githubusercontent.com/BlankSpruce/gersemi/master/gersemi/configuration.schema.json

definitions: []
disable_formatting: false
indent: 4
line_length: 80
list_expansion: favour-expansion
unsafe: false
warn_about_unknown_commands: false
@BlankSpruce
Copy link
Owner

AFAICT it was a bug already covered in #51. Try with the newest 0.18.2.

@BlankSpruce
Copy link
Owner

@b-nathan Let me know if the issue is cleared for you with the newest version.

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

No branches or pull requests

2 participants