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

UPDATE: align EIC Guide tasks with GitHub labels #279

Merged
merged 1 commit into from
Feb 28, 2024
Merged

Conversation

kierisi
Copy link
Contributor

@kierisi kierisi commented Feb 26, 2024

Based on internal conversations, I updated the instance of 1-editor-checks to 0/pre-review-checks, and added 1/editor-assigned to step four.

I also added two missing octothorpes to references to Slack channels.

Based on internal conversations, I updated the instance of `1-editor-checks` to ` 0/pre-review-checks `, and added `1/editor-assigned` to step four.

I also added two missing octothorpes to references to Slack channels.
@lwasser
Copy link
Member

lwasser commented Feb 28, 2024

@kierisi this is the first i'm hearing of the term octothorpe 😆 i looked it up. wherever did that word come from!? it might me my new favorite word

Copy link
Member

@lwasser lwasser 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 so so much for this @kierisi . i'll merge it now!

@lwasser lwasser merged commit ac67c1e into main Feb 28, 2024
4 checks passed
@lwasser
Copy link
Member

lwasser commented Feb 28, 2024

@all-contributors please add @kierisi for code, review

Copy link
Contributor

@lwasser

I've put up a pull request to add @kierisi! 🎉

@lwasser lwasser deleted the kierisi-patch-1 branch February 28, 2024 17:41
@kierisi
Copy link
Contributor Author

kierisi commented Feb 28, 2024

@lwasser Merriam-Webster says its origins are unknown! I came across a slide or blog post years ago that talked about how "octothorp" is the only globally-accepted term for "#" and so adopted the term from there!

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.

2 participants