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(next): better tsconfig #11859

Merged
merged 12 commits into from
Sep 3, 2024
Merged

feat(next): better tsconfig #11859

merged 12 commits into from
Sep 3, 2024

Conversation

florian-lefebvre
Copy link
Member

Changes

  • Update all relevant tsconfig
  • Remove env.d.ts
  • Do not create nor update env.d.ts anymore, recommend using tsconfig (see docs PR)

Testing

sync test updated

Docs

withastro/docs#9218

Copy link

changeset-bot bot commented Aug 28, 2024

🦋 Changeset detected

Latest commit: b2d1ea0

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added pkg: example Related to an example package (scope) pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) semver: major Change triggers a `major` release labels Aug 28, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This PR is blocked because it contains a major changeset. A reviewer will merge this at the next release if approved.

@florian-lefebvre florian-lefebvre changed the title feat: update code feat(next): better tsconfig Aug 28, 2024
Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Not quite the same, eh? 😅 Left you some thoughts!

.changeset/old-zebras-teach.md Outdated Show resolved Hide resolved
.changeset/old-zebras-teach.md Outdated Show resolved Hide resolved
Copy link
Member

@alexanderniebuhr alexanderniebuhr left a comment

Choose a reason for hiding this comment

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

We have moved the adapter to https://github.com/withastro/adapters, so this PR can't be merged here anymore.
I would offer to port over the changes to the new repo for you, just let me know if you want me to do that.

Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

A couple of tiny comments, but in general, I approve the changeset and trust whatever decisions you make are fine! 💪

florian-lefebvre and others added 3 commits September 2, 2024 15:20
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
@florian-lefebvre florian-lefebvre merged commit 3804711 into next Sep 3, 2024
14 checks passed
@florian-lefebvre florian-lefebvre deleted the feat/better-tsconfig branch September 3, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: astro Related to the core `astro` package (scope) pkg: example Related to an example package (scope) pkg: integration Related to any renderer integration (scope) semver: major Change triggers a `major` release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants