-
Notifications
You must be signed in to change notification settings - Fork 126
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
Fix / remove warnings in Masonry documentation #449
Comments
This is even worse with |
This is a start on linebender#449.
This is a start on linebender#449.
This is a start on #449.
This is work towards linebender#449.
This is work towards linebender#449.
This is work towards #449.
This should be fixed by #815. Are there still errors? |
This issue also tracks re-enabling checking for these issues in CI. Is that resolved? |
Well I didn't change the CI scripts after fixing the doc errors, so probably not. |
It looks like https://github.com/linebender/xilem/pull/815/files#diff-27b19fb95b25f32e39ea044b6ab68ee4a84d37944078b20ea52985fcca779e7aL112-L115 should have addressed this, I think. The only thing that's still wrong is
which is redundant I think. It looks like these were being suppressed by |
These are no longer suppressed above it in the crate, so this isn't needed now. Related to linebender#449.
These are no longer suppressed above it in the crate, so this isn't needed now. Related to #449.
Separate from #389, the warnings when running
cargo doc -p masonry
need to be fixed.Most of them are about things that no longer exist within masonry or have perhaps changed names, so it should be a pretty quick pass.
This is blocking re-enabling denying warnings within CI for xilem documentation and has already led to an issue in that regard with a broken link making its way into there.
The text was updated successfully, but these errors were encountered: