I assume you're going to push this to main, yes?

Yes before merging I'll make the main branch and make this PR go into main

@akeeste can you resubmit this PR against the new main branch?

Thanks @H0R5E! I reran the tests here and the documentation tests on WEC-Sim/WEC-Sim#1235. Everything seems to be passing now.

The last thing before merging #59 and WEC-Sim/WEC-Sim#1235, is to update the default branch on the Applications repo to main. If you can rename master directly to main, it looks like we'll have the added benefit of having API or web requests to master automatically redirect to the main branch https://github.com/github/renaming?tab=readme-ov-file#renaming-existing-branches

I'm not sure how renaming 'master' to 'main' will go if the 'main' branch already exists. If we need to take a couple extra steps to get the API / web requests automatically redirecting, it's worth it; e.g.

1. update this PR back into master so that it is not closed
2. delete the current main branch
3. then rename master to main with auto redirecting

@akeeste, I'm on it.

@akeeste, I found another reference two references to the master branch in the Wave_Markers folder. Can you fix this one, also?

@akeeste, I found another reference two references to the master branch in the Wave_Markers folder. Can you fix this one, also?

Good catch, done! I did one more search across all Applications repo files but did not find any other references to master. let me know if you catch anything else.

I also just confirmed that this repo's Mooring test does not checkout a specific master or main branch from the WEC-Sim/MoorDyn repo. There's only one branch and the default is always used for the test, so that repo should not affect this PR

@akeeste, I'm going to merge this now. Thanks for sorting this out and the tip about renaming the branches.