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.

Sign up for GitHub

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

Jump to bottom

fix typo #213

Closed
wants to merge 1 commit into from
Closed

fix typo #213

wants to merge 1 commit into from

Conversation

branfosj
Copy link
Member

No description provided.

@Micket
Copy link
Contributor

Micket commented Oct 17, 2023 •
edited
Loading

Also fix the typos that the CI complains about?
parallellisation ==> parallelisation

But fixing @akesandgren name is more difficult.

* GitHub: [@akesandgren](https://github.com/akesandgren) - e-mail: `ake.sandgren (at) hpc2n.umu.se` - IRC/Slack: `ake_s`

Check failure on line 83 in docs/maintainers.md
GitHub Actions / build

ake ==> ache

it shouldn't even be looking at code snippets, but something is slipping through. Either the email part, or the ake_s part

Workaround could be to just add ake to the list of ignored words?
https://github.com/easybuilders/easybuild-docs/blob/03891cbe6404a7fa237f289c99a660cfac5d7a73/.github/workflows/test.yml#L25C50-L25C59

@Micket Micket added this to the next release (4.8.2?) milestone Oct 17, 2023
@Micket
Copy link
Contributor

Micket commented Oct 18, 2023

I made all the fixes in #214

@Micket Micket closed this Oct 18, 2023
@branfosj branfosj deleted the specifying branch October 18, 2023 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.8.2
Development

Successfully merging this pull request may close these issues.
2 participants
@branfosj @Micket