Skip to content

tests: Add lsif_contains_generated_constant test #18309

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

Conversation

Urhengulas
Copy link
Contributor

For an upcoming open-source requirements-test-tracing tool cargo-needy we utilize the rust-analyzer lsif command to extract information about constants generated by a proc-macro. We would like to contribute a test which ensures that the command keeps containing that information.

We obviously would like the behavior we rely on to be kept, but we understand that it might change due to other goals. In this case we would kindly ask to inform us by opening a ticket at https://github.com/ferrocene/needy (not public yet, but soon!).

Note

Regarding the change in .gitignore: By removing the leading slash in the first line all target directories in the repository are gonna be ignored. If you would like to keep the previous version, I will have to add a .gitignore to the lsif-test-crate.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 16, 2024
@Urhengulas Urhengulas force-pushed the test-lsif_contains_generated_macros branch from 2a96039 to 31b11e0 Compare October 16, 2024 14:59
@Urhengulas Urhengulas changed the title tests: Add lsif_contains_generated_macros test tests: Add lsif_contains_generated_constant test Oct 16, 2024
@Urhengulas Urhengulas marked this pull request as draft October 18, 2024 10:32
@Urhengulas Urhengulas force-pushed the test-lsif_contains_generated_macros branch from babbd7e to 217ff18 Compare October 18, 2024 11:29
@Urhengulas Urhengulas force-pushed the test-lsif_contains_generated_macros branch from 217ff18 to cd5559f Compare October 18, 2024 11:31
@Urhengulas Urhengulas marked this pull request as ready for review October 18, 2024 11:31
@bors bors closed this in 4c4f186 Oct 22, 2024
lnicola pushed a commit to lnicola/rust that referenced this pull request Oct 29, 2024
…macros, r=lnicola

tests: Add `lsif_contains_generated_constant` test

Closes rust-lang/rust-analyzer#18309
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Nov 9, 2024
…=lnicola

tests: Add `lsif_contains_generated_constant` test

Closes rust-lang/rust-analyzer#18309
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants