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

Cleanup of formatting #190

Open
lazersos opened this issue Sep 7, 2023 · 0 comments
Open

Cleanup of formatting #190

lazersos opened this issue Sep 7, 2023 · 0 comments
Assignees
Labels
Announcement Official announcement to each user

Comments

@lazersos
Copy link
Collaborator

lazersos commented Sep 7, 2023

This issue is to announce that there will be cleanup of the formatting of the code. In particular, the general idea is to use fprettify to clean up the .f90 sources. The general notes on how to accomplish this are as follows:

  • Legacy .f and .f77 files should be left alone for now.
  • Each code/subdirectory should be handled by a separate branch for example cleanup/DIAGNO for the diagno code
  • Only whitespace formatting should be changed (no bugfixes of features added).
  • Code must compile with make clean_release
  • Any available tests must be conducted before a pull request is made.
  • GNU and Intel compilers must be checked
  • Pull requests should reference this issue Cleanup of formatting #190

Also if possible we'll convert the !DEC into proper C # pre-processor directives.

I (@lazersos) will first attempt this with DIAGNO then report back to this thread, at which point we can all start using the tool.

@lazersos lazersos added the Announcement Official announcement to each user label Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Announcement Official announcement to each user
Projects
None yet
Development

No branches or pull requests

2 participants