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

Docs: Maintenance > Bumping the Babel version #6027

Closed
2 tasks done
JoshuaKGoldberg opened this issue Nov 17, 2022 · 4 comments · Fixed by #7415
Closed
2 tasks done

Docs: Maintenance > Bumping the Babel version #6027

JoshuaKGoldberg opened this issue Nov 17, 2022 · 4 comments · Fixed by #7415
Assignees
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating

Comments

@JoshuaKGoldberg
Copy link
Member

JoshuaKGoldberg commented Nov 17, 2022

Before You File a Documentation Request Please Confirm You Have Done The Following...

Suggested Changes

Right now the @babel/* versions in our package.json are out of date, behind e.g. @babel/code-frame@7.18.6:

"@babel/code-frame": "^7.16.7",
"@babel/eslint-parser": "^7.18.2",
"@babel/parser": "^7.18.0",
"@babel/types": "^7.18.2",

I'm also seeing in fixtures-to-test.ts that there are some ignored fixture files due to previously filed Babel issues that have since been resolved:

* @see https://github.com/babel/babel/issues/14613

* @see https://github.com/babel/babel/issues/14589

When I update the @babel/* packages to latest, remove those ignores, and run typescript-estree tests, there are snapshot failures in typescript/expressions/instantiation-expression.src.ts. https://github.com/typescript-eslint/typescript-eslint/actions/runs/3490803456/jobs/5842786077

             91 |     // Received = Babel, Expected = TSESTree
           > 92 |     expect(babelAst).toEqual(tsestreeAst);

It'd be helpful to have the system of fixtures and snapshots documented. In particular I'd like to document what to do when these discrepencies show up in a package bump or AST fix.

#5885 is another example of where the docs would be helpful.

Edit: some context here too: #5717 (review), #5926 (comment)

Affected URL(s)

https://typescript-eslint.io/docs/maintenance/package-updates, maybe?

@JoshuaKGoldberg JoshuaKGoldberg added triage Waiting for maintainers to take a look documentation Documentation ("docs") that needs adding/updating labels Nov 17, 2022
@bradzacher
Copy link
Member

bradzacher commented Nov 18, 2022

ideally we should be pushing to move all of our fixture tests to ast-spec where all of this stuff is handled and reported automatically.

In a future state we would have all of the fixtures moved and then an upgrade should just be like:

$ yarn add -DW @babel/core @babel/parser @babel/eslint-parser
$ (cd packages/ast-spec && yarn test -u)
$ git commit -a -m "chore: bump babel dependency"

@JoshuaKGoldberg
Copy link
Member Author

JoshuaKGoldberg commented Nov 18, 2022

@bradzacher do you think it's reasonable to file a new issue to track those migrations? If so, is there a reasonable set of upgrade instructions/steps that can be put in it?

@bradzacher
Copy link
Member

Raised #6065

@JoshuaKGoldberg JoshuaKGoldberg added accepting prs Go ahead, send a pull request that resolves this issue and removed triage Waiting for maintainers to take a look labels Nov 24, 2022
@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Nov 24, 2022
@JoshuaKGoldberg JoshuaKGoldberg added the blocked by another issue Issues which are not ready because another issue needs to be resolved first label Dec 18, 2022
@JoshuaKGoldberg
Copy link
Member Author

Marking as blocked on #6065 so we don't spend time documenting the old flow.

@JoshuaKGoldberg JoshuaKGoldberg removed the blocked by another issue Issues which are not ready because another issue needs to be resolved first label Jul 4, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepting prs Go ahead, send a pull request that resolves this issue documentation Documentation ("docs") that needs adding/updating
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants