Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix spelling #988

Merged
merged 4 commits into from
Jan 15, 2025
Merged

Conversation

sjspielman
Copy link
Member

Closes #987

This PR fixes typos:

  • I added words to dictionary
  • I fixed a bad "neighborhood" spelling in hello-clusters and re-rendered the notebook
  • I added lots of things in ewings/references/README.md into backticks to not throw a spelling error
    • While I was modifying this file, I noticed a line at the end that didn't seem like it was meant to be there but might have been left over from copy/pasting when creating this bullet list? It's a line that just contains text (without a bullet point) GO:BP ECM Organization. I deleted the line, but let me know if it was supposed to be a bullet in its own right or should be something else altogether?
  • I also took this opportunity to update the issue template for monthly spellchecks; after writing the template, we decided not to run the action to confirm spelling errors were fixed, so I removed that essentially "deprecated" bullet

@sjspielman sjspielman requested review from allyhawkins and removed request for jaclyn-taroni January 15, 2025 15:12
@sjspielman
Copy link
Member Author

I canceled the ewings job - no need to spend cycles for this module here when only a README was modified.

Copy link
Member

@allyhawkins allyhawkins left a comment

Choose a reason for hiding this comment

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

LGTM

@sjspielman sjspielman merged commit a27ab31 into AlexsLemonade:main Jan 15, 2025
4 of 5 checks passed
@sjspielman sjspielman deleted the sjspielman/987-spellcheck branch January 15, 2025 15:39
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.

Monthly spellcheck results
2 participants