docs (epic): overview of documentation changes #1754
Labels
type: fix
Issues describing a broken feature.
Comments
braaar
added
status: needs triage
|
I think this issue has outlived its usefulness by now |
braaar
removed
the
status: needs triage
|
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
The goal of this issue is to have an easy to digest overview over docs changes to make it easier for maintainers to go through simple documentation changes.
Isn't this overview unnecessary since you can just filter the PRs for
docs?Not quite, since not everyone uses correct PR titles, and I think it helps that I have gone over the PRs and given them a quick look before I put them in the "ready for maintainer review" list. I've also tried to identify duplicates.
Where are the docstring changes?
Since docstrings are published as a part of this module, I think it makes sense to consider them as part of the product itself, whereas the documentation in the readme here in the repo is "just" documentation. I have chosen to put docstring changes as bugfixes in #1751
Proposed solution
Review and merge the open PRs that fix bugs
Ready for maintainer review
@IsBooleanStringdecorator #1655validationOptionscan not be undefined" when creating custom class validators #1643Need some work still (mostly just rebase/merge)
forbidUnknownValues#1355Don't have a PR yet
(comment if you know any other simple issues that should be tracked here!)
I'll try to look through issues to see if I can spot some.
@jotamorais
The text was updated successfully, but these errors were encountered: