Add warning for missing documentation #727
沒有審核者
標籤
未選擇標籤
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
沒有里程碑
未選擇專案
沒有負責人
2 位參與者
通知
截止日期
未設定截止日期。
先決條件
未設定先決條件。
參考: Plume/Plume#727
載入中…
新增表格
新增問題並參考
沒有提供敘述。
刪除分支「missing-docs」
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
To force us to document our code.
I didn't added it to binaries (
src/
andplume-cli/
because it didn't really made sense for them IMO), but I can.Also, to keep this PR small, I propose that we slowly document each function as we modify them in other PRs. Does it seem reasonable to you?
The overall idea sounds reasonable. Considering
src/
(maybeplume-cli
) also contains some non trivial code (remember url! in the custom domain PR?), I think doing it would make sense to apply that on them tooYou should add a feature to all crates, and replace current attribute with
#![cfg_attr(not(feature = "ci"), warn(missing_docs))]
(which would be set in CI). That way it won't fail on the deny(warnings) on clippycan't we add that that somewhere to
clippy
orcheck
?Isn't there a better way to allow just these warnings in clippy?
Codecov Report
合併請求已關閉