-
Notifications
You must be signed in to change notification settings - Fork 33
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/whitespace and formatting #1049
Fix/whitespace and formatting #1049
Conversation
f6fab3c
to
25c1005
Compare
The force push was a rebase onto the newest develop branch. |
Documentation now includes the options for findent which reproduce the target formatting.
2f8f1e8
to
1dd77db
Compare
Nice! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, however not sure what todo with the legacy speclib
file, which the linter has a hard time with
This reverts commit b8b2c10.
- Currently it only reports if the format is satisfied
Please note that i have added a Formatting CI rule. (I also have never tried doing something like that, but i found a few stack overflow pages mentioning it.) |
This is more a suggestion of how to handle formatting in a unified way.
findent was used to reformat all files according to the code style guide.
findent can be integrated directly into Vim, Emacs and GEdit.
In VSCode the ModernFortran package allow the use of "findent"