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

schema/lang: Rename Traversal* to Reference* #266

Closed
3 tasks
radeksimko opened this issue Apr 25, 2023 · 0 comments · Fixed by #362
Closed
3 tasks

schema/lang: Rename Traversal* to Reference* #266

radeksimko opened this issue Apr 25, 2023 · 0 comments · Fixed by #362
Assignees

Comments

@radeksimko
Copy link
Member

radeksimko commented Apr 25, 2023

Depends on

Context

As part of #185 and some other related PRs we started using a slightly different naming convention (traversal -> reference essentially).

We used the term "reference" in other places before (there's whole package reference), so it makes sense to align the rest of the codebase and get rid of "traversal", which is usually only how we refer to references in HCL internals. The aim is to make the public API friendlier to developers who have to build the schema.

Screenshot 2023-04-25 at 14 05 16

Proposal

  • Rename all existing types/functions which involve "traversal" to "reference".
  • Update any references in terraform-schema
  • Update any references in terraform-ls
@radeksimko radeksimko changed the title schema: Rename Traversal* to Reference* schema/lang: Rename Traversal* to Reference* Apr 25, 2023
@dbanck dbanck self-assigned this Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants