nextsimhub / nextsimdg

neXtSIM_DG : next generation sea-ice model with DG

https://nextsim-dg.readthedocs.io/en/latest/?badge=latest

Apache License 2.0

10 stars 13 forks source link

Format dynamics code #617

Closed TomMelt closed 4 months ago

TomMelt commented 4 months ago

Format Dynamics Code

This PR brings the dynamics folder inline with the rest of the nextsimdg code.

[x] rename dynamics/tests folder to dynamics/test
[x] clang-format dynamics source

NOTE: These changes are formatting only. There are no changes to the actual logic of the source code

This will serve as the base for #556 which reinstates the dynamics tests

TomMelt commented 4 months ago

(copying from #556 )

my plan is not to make any additional changes to [this PR], for the main reason I want it to just be changes relating to clang format and the renaming of the tests folder. Happy to make a separate PR to deal with the changes you suggested, but I don't want to add any additional confusion to an already tricky PR :sweat_smile:

timspainNERSC commented 4 months ago

(copying from #556 )

my plan is not to make any additional changes to [this PR], for the main reason I want it to just be changes relating to clang format and the renaming of the tests folder. Happy to make a separate PR to deal with the changes you suggested, but I don't want to add any additional confusion to an already tricky PR 😅

👍 Completely understandable!