-
Notifications
You must be signed in to change notification settings - Fork 6
Intl documentation did not generate #46
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
Comments
Watching, because I'd like to know what we should do when doc generation doesn't work. |
In this case I go to https://console.cloud.google.com, go to Datastore, and remove the intl 0.15.0 entity from the Datastore. Then, unfortunately, I have to kill the instance that runs dartdoc generator, because the intl package is still cached there. So, I go to GCE instance groups, and change the size of |
Wow. Why do you think intl doc generation failed? |
Here's the log: https://www.dartdocs.org/documentation/intl/0.15.0/log.txt It says |
Perhaps a pub.dartlang.org failure? |
Maybe. Would be probably nice to write some script, that'd automate that, but no such script exists yet :) |
cc @isoos, @mkustermann |
Kind of related: #34 |
If it works now, then it seems likely to have been the tagging issue. I don't see docs for 0.15.0 on the site, but I don't know if that just means it hasn't tried again yet. I notice that with the failure logs they show the time, but not the time zone or the date, so I have no way of telling if that's the old log or it gave the same error again. |
Hmm, now it fails with the different error: https://www.dartdocs.org/documentation/intl/0.15.0/log.txt |
So that seems to be a bug in dartdoc, introduced 5 days ago in dart-lang/dartdoc@7764f87 It's reporting some code that's been in Intl for years, where both date_symbol_data_file and date_symbol_data_http re-export a list from another library, and it considers that ambiguous. But those two libraries can never be imported at the same time, since one depends indirectly on dart:html and the other on dart:io. |
Sorry, wrong commit. It was dart-lang/dartdoc@47b60b4 |
Actually, it kind of looks like it's split between the two. Anyway, this really doesn't seem like it should refuse to produce documentation even if there was an ambiguity. |
Try using 0.11.2 dartdoc, just published. |
Just checking in; does 0.11.2 dartdoc work better to correct this issue? |
No idea, haven't tried yet. |
Could we try. This is an important package, it would be good to have documentation. I notice that intl_translation also doesn't have docs, but it might be a different reason, there's no entry in the build failures. |
https://www.dartdocs.org/documentation/intl/latest now works (redirecting to https://www.dartdocs.org/documentation/intl/0.15.1/index.html). |
See https://www.dartdocs.org/documentation/intl/0.15.0/
It might have been because the 0.15.- release wasn't tagged, but now that it is, I don't know how to kick it to try again.
The text was updated successfully, but these errors were encountered: