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

Only filter out duplicates if they are of the same kind #781

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bangbangsheshotmedown
Copy link

No description provided.

Copy link

Thanks for your Pull Request and making D better!

This comment will automatically be updated to summarize some statistics in a few minutes.

@WebFreak001
Copy link
Member

missing test to indicate what exactly this fixes or changes

@bangbangsheshotmedown
Copy link
Author

bangbangsheshotmedown commented Nov 17, 2024

From what i can remember, it was not showing a variable if its name was the same of a struct for example, but i'm not sure anymore

I will try to write a test, but i'm not sure how yet

@WebFreak001
Copy link
Member

WebFreak001 commented Nov 29, 2024

just create a folder in the tests folder that contains a script to run a test. You can find the most basic example here:

https://github.com/dlang-community/DCD/tree/master/tests/tc001

Run dcd-client with the -X flag to include a lot more information that you can test for

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants