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

refactor(semantic)!: rename AstNode to Node and AstNodes to Nodes #5767

Conversation

overlookmotel
Copy link
Contributor

@overlookmotel overlookmotel commented Sep 13, 2024

#5740 renamed AstNodeId to NodeId. To match this, also rename:

  • AstNode -> Node
  • AstNodes -> Nodes

Copy link

graphite-app bot commented Sep 13, 2024

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.

@github-actions github-actions bot added A-linter Area - Linter A-semantic Area - Semantic labels Sep 13, 2024
Copy link
Contributor Author

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

Join @overlookmotel and the rest of your teammates on Graphite Graphite

@overlookmotel overlookmotel force-pushed the 09-13-refactor_semantic_rename_astnode_to_node_and_astnodes_to_nodes_ branch from 75c81ae to bf388d9 Compare September 13, 2024 15:28
@overlookmotel
Copy link
Contributor Author

@Boshen I think it makes sense for consistency, but is it a step too far?

Copy link

codspeed-hq bot commented Sep 13, 2024

CodSpeed Performance Report

Merging #5767 will not alter performance

Comparing 09-13-refactor_semantic_rename_astnode_to_node_and_astnodes_to_nodes_ (e7efaff) with main (aac8316)

Summary

✅ 29 untouched benchmarks

@overlookmotel overlookmotel force-pushed the 09-13-refactor_semantic_rename_astnode_to_node_and_astnodes_to_nodes_ branch from bf388d9 to 1fa3381 Compare September 13, 2024 16:55
@overlookmotel overlookmotel force-pushed the 09-13-refactor_semantic_rename_astnode_to_node_and_astnodes_to_nodes_ branch from 1fa3381 to e7efaff Compare September 13, 2024 16:56
@overlookmotel overlookmotel marked this pull request as ready for review September 13, 2024 17:01
@overlookmotel
Copy link
Contributor Author

I changed my mind. I don't think this is a good idea. We also have AstKind and AstType, which we don't want to rename to just Kind and Type, so Ast* is going to remain a thing.

We plan to remove the AstNode type anyway as part of #5689. Once that's done, we could rename AstNodes to Nodes.

@Boshen Boshen deleted the 09-13-refactor_semantic_rename_astnode_to_node_and_astnodes_to_nodes_ branch February 17, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter A-semantic Area - Semantic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant