feat(ast): provide NONE type for AST builder calls

[overlookmotel]

Closes #5736.

Introduce a NONE type which can be used for any AstBuilder method call param which expects an IntoIn<'a, Something<'a>>, where otherwise you have to provide a verbose type annotation.

Before:

ast.arrow_function_expression(
    SPAN,
    is_expression,
    is_async,
    None::<TSTypeParameterDeclaration>,
    params,
    None::<TSTypeAnnotation>,
    body,
)

After:

ast.arrow_function_expression(SPAN, is_expression, is_async, NONE, params, NONE, body)

[overlookmotel]

• feat(ast): provide NONE type for AST builder calls #5737 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #5737 will not alter performance

_{Comparing 09-12-feat_ast_provide_none_type_for_ast_builder_calls (953fe17) with main (71116a1)}

Summary

✅ 29 untouched benchmarks

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[rzvxa]

Sorry, I forgot about this issue😓 Thanks for addressing it.

[graphite-app]

Merge activity

• Sep 12, 10:39 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Sep 12, 10:39 PM EDT: Boshen added this pull request to the Graphite merge queue.
• Sep 12, 10:46 PM EDT: Boshen merged this pull request with the Graphite merge queue.