eslint.org icon indicating copy to clipboard operation
eslint.org copied to clipboard

feat: blog: Code review nit to ecosystem improvements

Open captbaritone opened this issue 1 year ago • 9 comments

Prerequisites checklist

What is the purpose of this pull request?

Add a blog post reflecting on how ESLint helped a small nit I noticed in code review grow into a fundamental ecosystem improvement shipping in the newest version of TypeScript.

What changes did you make? (Give an overview)

Added a blog post

Related Issues

I previously submitted a blog post about this rule: https://github.com/eslint/eslint.org/pull/240/files

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

captbaritone avatar Sep 17 '24 22:09 captbaritone

Deploy Preview for new-eslint ready!

Built without sensitive environment variables

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/new-eslint/deploys/66fd8c80944c770008bc438f
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 22:09 netlify[bot]

Deploy Preview for hi-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/hi-eslint/deploys/66fd8c80ff999d0008c0f0c9
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 22:09 netlify[bot]

Deploy Preview for ja-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/ja-eslint/deploys/66fd8c800434c60008fefc8a
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 22:09 netlify[bot]

Deploy Preview for zh-hans-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/zh-hans-eslint/deploys/66fd8c802291080008b44f1a
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

Deploy Preview for fr-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/fr-eslint/deploys/66fd8c80bf5d30000800d5a2
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

Deploy Preview for es-eslint ready!

Name Link
Latest commit ebd41674dd3af2dd223c4dcc31142321fd96154e
Latest deploy log https://app.netlify.com/sites/es-eslint/deploys/66ea094bc8871e00088e0fe9
Deploy Preview https://deploy-preview-629--es-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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

Deploy Preview for pt-br-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/pt-br-eslint/deploys/66fd8c80aeb826000814a303
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

Deploy Preview for de-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/de-eslint/deploys/66fd8c8075dc4a00080fcdea
Deploy Preview https://deploy-preview-629--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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

Deploy Preview for es-eslint ready!

Name Link
Latest commit 5e85697a0cd2cebd4128a7b78b70b328bdaa45fb
Latest deploy log https://app.netlify.com/sites/es-eslint/deploys/66fd8c80aec24c0008ea82d5
Deploy Preview https://deploy-preview-629--es-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 configuration.

netlify[bot] avatar Sep 17 '24 23:09 netlify[bot]

@captbaritone just checking in on this. It looks like there's some feedback that hasn't been addressed. Do you have a timetable for revisiting?

nzakas avatar Sep 30 '24 20:09 nzakas

@nzakas Thanks for the ping, I had a bit of writers block on the new conclusion, but I've reworked it and am now quite happy with it. Curious to hear any further feedback you have on this most recent draft.

captbaritone avatar Oct 02 '24 06:10 captbaritone

Sounds good, and thanks for all your feedback, it has greatly improved the post.

captbaritone avatar Oct 02 '24 18:10 captbaritone