Skip to content

Rename master to main - #378

Open
fletchapin wants to merge 7 commits into
mainfrom
master
Open

Rename master to main#378
fletchapin wants to merge 7 commits into
mainfrom
master

Conversation

@fletchapin

Copy link
Copy Markdown
Member

Pull request recommendations:

  • Name your branch/PR your-development-type/short-description. Ex: feature/read-tiff-files
  • Link to any relevant issue in the PR description. Ex: Resolves [running doctests automagically with Sphinx #12], adds tiff file format support
  • Provide context of changes.
  • Provide relevant tests for your feature or bug fix.
  • Provide or update documentation for any feature added by your pull request.

I went to rename the default branch to main and noticed that main already existed, but diverged from master. I propose that we merge master into main and then set main as the default branch moving forward.

@fletchapin
fletchapin requested a review from monroews September 16, 2025 17:31
@monroews

monroews commented Sep 16, 2025

Copy link
Copy Markdown
Collaborator

Are there any changes to textbook content with these changes? I'd like to start with the textbook as it currently is published and then verify that these changes are actually beneficial.

The entrance tank is currently published. https://aguaclara-reach.github.io/Textbook/Flow_Control_and_Measurement/ET_Design_Challenge.html

I reread what you are doing and I think it is okay. These are changes are needed so that main is up to date.

Corrected the dimensions of energy dissipation rate.
@fletchapin

fletchapin commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

Looking at the commit history, these are changes that you and Ethan made last year but did not get into the main branch, so I think I was just trying to manage the Git branches and not change any content.

@monroews sorry I didn't see your comment for months!

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.

4 participants