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

feat: blog: ESLint Community org announcement #402

Merged

Conversation

MichaelDeBoey
Copy link
Contributor

Prerequisites checklist

What is the purpose of this pull request?

Adding a blogpost to announce the official ESLint Community org

What changes did you make?

I added a blogpost to announce the official ESLint Community org

Related Issues

N/A

Is there anything you'd like reviewers to focus on?

Spelling, grammar & if everything's included that @nzakas wants to include.

@netlify
Copy link

netlify bot commented Jan 10, 2023

👷 Deploy request for es-eslint pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 57874f1

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for hi-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/hi-eslint/deploys/641909f2f089e20008883fb7
😎 Deploy Preview https://deploy-preview-402--hi-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for ja-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/ja-eslint/deploys/641909f20db78e000789660b
😎 Deploy Preview https://deploy-preview-402--ja-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for new-eslint ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/new-eslint/deploys/641909f26197100008cd72cf
😎 Deploy Preview https://deploy-preview-402--new-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for fr-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/fr-eslint/deploys/641909f2faf61900080d3b35
😎 Deploy Preview https://deploy-preview-402--fr-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for pt-br-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/pt-br-eslint/deploys/641909f2fec3ba0008c6bece
😎 Deploy Preview https://deploy-preview-402--pt-br-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for de-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/de-eslint/deploys/641909f2e68a7400087de368
😎 Deploy Preview https://deploy-preview-402--de-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for zh-hans-eslint ready!

Name Link
🔨 Latest commit 57874f1
🔍 Latest deploy log https://app.netlify.com/sites/zh-hans-eslint/deploys/641909f2a86b4a000823837d
😎 Deploy Preview https://deploy-preview-402--zh-hans-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch 4 times, most recently from 4b70717 to 11aa924 Compare January 10, 2023 16:52
@amareshsm amareshsm added accepted and removed triage labels Jan 10, 2023
@amareshsm amareshsm closed this Jan 10, 2023
@amareshsm amareshsm reopened this Jan 10, 2023
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
@amareshsm
Copy link
Member

CI is failing because the author info is not available. @MichaelDeBoey pls add your details in all_authors.json

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together. I cleaned up some of the wording and left some other comments.

Please move this into the drafts folder, as we aren't sure when this will be published yet.

Also, this file isn't building correctly, all of the CI is failing. Please double-check that.

src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
src/content/blog/2023-01-10-eslint-community-org.md Outdated Show resolved Hide resolved
@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch 3 times, most recently from d097c84 to 155f68f Compare January 11, 2023 01:20
@MichaelDeBoey MichaelDeBoey requested review from bmish and nzakas and removed request for bmish January 11, 2023 01:35
@MichaelDeBoey MichaelDeBoey requested review from bmish and nzakas and removed request for bmish January 11, 2023 16:28
@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch 5 times, most recently from 477c031 to b1c19ba Compare January 14, 2023 00:55
Copy link
Member

@kecrily kecrily left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

Thanks for cleaning this up, it's looking a lot better. I've left some minor stylistic suggestions (ampersands shouldn't be used in prose) and some other general comments. I think we want to be careful to put our best foot forward with this announcement and be clear on the role of eslint-community. I left some comments to that effect.

src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch from b1c19ba to e12ab91 Compare January 16, 2023 21:31
@amareshsm amareshsm requested a review from nzakas January 23, 2023 12:33
Copy link
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

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

LGTM. Waiting for @nzakas approval.

@nzakas
Copy link
Member

nzakas commented Feb 7, 2023

@MichaelDeBoey can you please update the blog post? Let me know if you have any questions about my last round of feedback. Thanks!

@nzakas
Copy link
Member

nzakas commented Mar 8, 2023

@MichaelDeBoey just checking in again. I'd really like to get this finished and posted. Can you update it please?

@MichaelDeBoey
Copy link
Contributor Author

@nzakas I've been away for some time, so sorry for the late response.

I'll try to take some time later this week to look into updating the blogpost.

@nzakas
Copy link
Member

nzakas commented Mar 14, 2023

@MichaelDeBoey sounds good. thanks!

@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch from e12ab91 to c7f424c Compare March 14, 2023 22:13
Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

Thanks for this. I left some suggestions on how we can further clarify things.

We still need a conclusion for this post, as well.

Thanks! We are close!

src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
src/content/drafts/eslint-community-org.md Outdated Show resolved Hide resolved
@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch 3 times, most recently from d61970a to 771af00 Compare March 16, 2023 01:25
@MichaelDeBoey MichaelDeBoey force-pushed the community-org-announcement-blogpost branch from 771af00 to 57874f1 Compare March 21, 2023 01:35
Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

Thanks. I'm going to clean up some of the formatting and then I'll publish it.

@nzakas nzakas merged commit 3af313e into eslint:main Mar 21, 2023
@MichaelDeBoey MichaelDeBoey deleted the community-org-announcement-blogpost branch March 21, 2023 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants