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

Unified Style Specification #7

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

Conversation

IogaMaster
Copy link

No description provided.

@IogaMaster IogaMaster changed the title Style RFC: Initial Unified Style Specification Sep 29, 2023
@IogaMaster
Copy link
Author

cc/ @jntrnr @AucaCoyan @amtoine

@amtoine
Copy link
Member

amtoine commented Sep 30, 2023

we have two documents that might be worth a look

@AucaCoyan
Copy link

Hi! Thanks to pull up a RFC, I think we have to clear an delimit a goal for this specific document, otherwise we can discuss a lot of topics abroad a lot of time!

AucaCoyan mentioned that there is no standardization of style

Not really (if I did, sorry, I was wrong!), I said that there are multiple gears working on the area LSP, parser, formatter, editor-support and so on. and I'm not sure if we all are onboard with the same info and objectives.
We may have different projects, and that's okay! We only need to focus the same goals in the same offices, so to speak.
We do have stylistic recommendations, as amtoine pointed out.

Refering to this issue topic, the current question I ask is:

  • Is it necessary to upgrade the syle guide it to a better version? what is the purpose of those documents?
    Do you seek a different purpose @IogaMaster ?

@IogaMaster
Copy link
Author

Do you seek a different purpose

No, I just would like the style guide to reach more eyes, as nufmt doesn't get a lot of changes.

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.

None yet

3 participants