Skip to content

Conversation

@etiennenoel
Copy link
Collaborator

@etiennenoel etiennenoel commented Apr 6, 2025

Here's a list of things this PR does:

  • Upgrade to Typescript 5.8.3
    • Adds support for es2020 and esnext
    • Updates tslib to latest
  • Upgrade eslint + prettier to latest version
    • Migrates eslint config to js file
    • Fixes linting in entire project to match latest rules
  • Removes support for NodeJS lower than 18 (they are unsupported)

Not sure if you want to merge it now, but this gives us an idea of what needs to happen to update the dependencies. I feel like updating the dev dependencies is useful.

However, this also updates tslib and I need to understand better what the implications of doing that are. Does that mean we should do a major upgrade? If yes, then I can start planning about features we would want to include if we decide to go down that route.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 24 out of 26 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • .eslintrc.yml: Language not supported
  • package.json: Language not supported

@etiennenoel etiennenoel marked this pull request as draft April 6, 2025 03:36
@etiennenoel etiennenoel removed the request for review from Xapphire13 April 6, 2025 03:36
@etiennenoel etiennenoel marked this pull request as ready for review April 6, 2025 03:53
@etiennenoel etiennenoel requested a review from Xapphire13 April 6, 2025 03:54
@etiennenoel
Copy link
Collaborator Author

@Xapphire13 This is just an idea of upgrading the dependencies and removing support for older NodeJS versions. I don't think I would merge it now, but if we move towards a new major version, maybe this is part of this new version? Let me know your thoughts and I can also start preparing some ideas for this next major version.

@MeltingMosaic
Copy link
Collaborator

I think this would be a major version upgrade. There's a risk of breaking changes with older TS versions. @Xapphire13 and I discussed this a few years back and I recall we decided that if we did get to doing full TS5 support we'd up the major version.

@etiennenoel
Copy link
Collaborator Author

I agree. I think it's worth planning this properly. Then, what I think I can do, is not to merge this on master but merge this in a 5.x.x branch so we can see how this project could evolve and then decide if it makes sense or not to continue.

@etiennenoel etiennenoel changed the base branch from master to 5.x.x April 6, 2025 04:14
@etiennenoel etiennenoel merged commit 1b528af into microsoft:5.x.x Apr 6, 2025
1 check 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.

2 participants