Skip to content

[Question] Log the warnings #3391

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.

Sign up for GitHub

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

Jump to bottom
Closed
HeySreelal opened this issue Apr 19, 2023 · 3 comments
Closed

[Question] Log the warnings #3391

HeySreelal opened this issue Apr 19, 2023 · 3 comments
Labels
P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@HeySreelal
Copy link

I've run Dart doc and the results included this line:

Found 7 warnings and 0 errors.

But, no matter what flag I pass I'm not logged with the actual warnings that are occurring.

I tried dart doc --verbose, I tried to use the dartdoc globally and used --verbose-warnings, and --warnings. No useful information was found.

Is there a way that I can get logs on where in my code actually that warning appears?
@jcollins-g jcollins-g added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) labels Apr 19, 2023
@jamesderlin
Copy link

Also see #3331.

Warnings are kind of useless if there's no way to see what they are.

@jcollins-g jcollins-g mentioned this issue Apr 27, 2023
Merged
7 tasks
auto-submit bot pushed a commit to flutter/flutter that referenced this issue Apr 27, 2023
@jcollins-g
Update dartdoc to v6.2.2. (#125650)
fb36bba 
This PR updates the dartdoc version to v6.2.2.

Release notes:  https://github.com/dart-lang/dartdoc/releases/tag/v6.2.2

The main impact for Flutter is to update dartdoc to add chips to class and/or mixin pages for class modifiers.  See dart-lang/dartdoc#3391, dart-lang/dartdoc#3402.   This will also be a cherry-pick candidate for 3.1 if it can land in time.

- [ test-exempt ] I added new tests to check the change I am making, or this PR is [test-exempt].
jcollins-g added a commit to flutter/flutter that referenced this issue Apr 28, 2023
@jcollins-g
Update dartdoc to v6.2.2. (#125650)
f9240aa 
This PR updates the dartdoc version to v6.2.2.

Release notes:  https://github.com/dart-lang/dartdoc/releases/tag/v6.2.2

The main impact for Flutter is to update dartdoc to add chips to class and/or mixin pages for class modifiers.  See dart-lang/dartdoc#3391, dart-lang/dartdoc#3402.   This will also be a cherry-pick candidate for 3.1 if it can land in time.

- [ test-exempt ] I added new tests to check the change I am making, or this PR is [test-exempt].
@srawlins
Copy link
Member

Sorry for the extreme delay!

I have seen this once before, and I think I worked around it somehow, but I've lost the knowledge. IIRC, the warnings were about broken links or orphaned files or something... But yes, super frustrating that they are counted, but not printed.

If you have a reproduction, we can look into it. (Or the next time I can repro it myself I can investigate properly.)

@HeySreelal
Copy link
Author

Hi @srawlins, I've tried to reproduce the issue with no success. Also, found the warnings are now being logged to the console. Which pretty much sounds like we can close this issue. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@srawlins @jcollins-g @jamesderlin @HeySreelal