Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Typo in the README directory for ECCO errors #343

Merged
merged 2 commits into from
Feb 20, 2025
Merged

Conversation

taimoorsohail
Copy link
Contributor

@taimoorsohail taimoorsohail commented Feb 20, 2025

Hi, I am just trying to run a ClimaOcean example and noticed there is an issue with the error message pointing people to the README file which tells you how to get your ECCO username and password. I have fixed it.

Sorry not sure how the gitignore file snuck in there but it hasn't had changes made to it.

Copy link
Member

@glwagner glwagner left a comment

Choose a reason for hiding this comment

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

thank you!

@glwagner glwagner merged commit 459d76d into CliMA:main Feb 20, 2025
0 of 5 checks passed
@navidcy
Copy link
Collaborator

navidcy commented Feb 21, 2025

welcome to ClimaOcean contributors!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Just an attempt to reduce the wonder why gitignore appeared to have changed. Seems like you deleted an empty line in the end. Sometimes our editor might be doing this automatically (similar for trailing spaces) because of a setting/an extension.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants