Skip to content

Conversation

jeis4wpi
Copy link

Propose spelling fix

Copy link
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look fine.

@pstaabp
Copy link
Member

pstaabp commented Oct 7, 2025

@jeis4wpi In the future, name the branch that you want to merge in something different than develop. Generally, we like to test the code and having the branch called develop makes it more difficult. For example, I would call this branch fix-spelling or something similar.

@jeis4wpi
Copy link
Author

jeis4wpi commented Oct 7, 2025

Thank you for the advice @pstaabp

Copy link
Member

@drgrice1 drgrice1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the code formatting issue, this looks good.

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