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

Track line number additionally to file name for inclusion reason #209

Open
janpio opened this issue May 18, 2021 · 2 comments
Open

Track line number additionally to file name for inclusion reason #209

janpio opened this issue May 18, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@janpio
Copy link

janpio commented May 18, 2021

Would it be possible for nft to track the line number that caused it to include something in the tree vs. just the file/parent? Then it would be trivial to look at the correct code to figure out what lead to an inclusion in the archive.

(Extracted from #203 (comment))

@styfle
Copy link
Member

styfle commented May 18, 2021

Sounds like a nice debugging feature. Do you want to submit a PR to add it? 🙂

@styfle styfle added the enhancement New feature or request label May 18, 2021
@janpio
Copy link
Author

janpio commented May 19, 2021

Right now no capacity (surprise...), but if I have a future reason to actually jump into the codebase to understand it I will certainly remember this enhancement request here and take a shot. Anyone else of course feel free to do so in the meantime.

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

No branches or pull requests

2 participants