Skip to content

Conversation

@InertSloth
Copy link
Contributor

@InertSloth InertSloth commented Sep 14, 2025

Description (required)

tseslint.config is deprecated and emits a warning, this changes to 9.x defineConfig() format, in this case should be backwards compatible.

Related issues & labels (optional)

Many astro users will peek at the official docs repo to see how they do things. So other astro projects may model after this eslint.config.mjs so it behooves us to make it pass as cleanly as possible. astro docs wether it wants to be or not will have the first time perception of being the "shining example" for how to do things.

First time contrib: Yes for PRs.

discord: inert.sloth

- 🦥

tseslint.config is deprecated and emits a warning, this changed to 9.x defineConfig() format, in this case should be backwards compatible.
@netlify
Copy link

netlify bot commented Sep 14, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cee4303
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68c71caf7188d700089d5beb
😎 Deploy Preview https://deploy-preview-12315--astro-docs-2.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 project configuration.

@astrobot-houston
Copy link
Contributor

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any broken links you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

Copy link
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

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

Thanks @InertSloth!

I dug up the migration guide, just to double check this and it looks good 👍

@delucis delucis merged commit aa21acd into withastro:main Sep 15, 2025
11 checks passed
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.

3 participants