-
Notifications
You must be signed in to change notification settings - Fork 160
Upgrade dartdoc to 0.21.1 #1650
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@@ -4,7 +4,7 @@ | |||
<meta charset="utf-8"> | |||
<meta http-equiv="X-UA-Compatible" content="IE=edge"> | |||
<meta name="viewport" content="width=device-width, initial-scale=1"> | |||
<meta name="generator" content="made with love by dartdoc 0.20.4"> | |||
<meta name="generator" content="made with love by dartdoc 0.21.1"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we mock out the dartdoc version so updating it doesn't change this file unless the output format actually changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather keep it here, because updating the customization may not be at the same time with the update of the dartdoc version.
@@ -87,12 +87,12 @@ <h3 id="usage">Usage</h3> | |||
<h2>Libraries</h2> | |||
<dl> | |||
<dt id="models"> | |||
<span class="name"><a href="models/models-library.html">models</a></span> | |||
<span class="name"><a href="models/models-library.html">models</a></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the trailing whitespace be reported as a bug in dartdoc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reported: dart-lang/dartdoc#1762
#1645
From our perspective the generated dartdoc didn't change much.