🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Add `codespell` to the CI (#72)
We need to add
Example:
• https://github.com/dry-python/returns/blob/1d74c969a94ce13648be2fe2a8bfad3b12f73069/pyproject.toml#L64
• Configuration (should be moved to
• Running: https://github.com/dry-python/returns/blob/1d74c969a94ce13648be2fe2a8bfad3b12f73069/.github/workflows/test.yml#L61
PRs are welcome!
#enhancement #good_first_issue #github_actions #help_wanted
sent via relator
📝 Add `codespell` to the CI (#72)
We need to add
codespell
to the CI, because I make a lot of typos :)Example:
• https://github.com/dry-python/returns/blob/1d74c969a94ce13648be2fe2a8bfad3b12f73069/pyproject.toml#L64
• Configuration (should be moved to
pyproject.toml
): https://github.com/dry-python/returns/blob/1d74c969a94ce13648be2fe2a8bfad3b12f73069/setup.cfg#L170-L173• Running: https://github.com/dry-python/returns/blob/1d74c969a94ce13648be2fe2a8bfad3b12f73069/.github/workflows/test.yml#L61
PRs are welcome!
#enhancement #good_first_issue #github_actions #help_wanted
sent via relator
❤1👍1
🚀 New issue to wemake-services/wemake-python-styleguide by @sobolevn
📝 `WPS226` false positive on fstring parts (#3548)
This code raises a violation:
When
Output:
This does not seem right, because strings that are part of
PR would be rather easy :)
#help_wanted #levelstarter #good_first_issue #bug
sent via relator
📝 `WPS226` false positive on fstring parts (#3548)
This code raises a violation:
x = f'single {1}'
y = f'single {1}'
When
--max-string-usage=1
is provided.Output:
1:5 WPS226 Found string literal over-use: single > 1
x = f'single {1}'
^
This does not seem right, because strings that are part of
f
strings should not be counted by WPS226
PR would be rather easy :)
#help_wanted #levelstarter #good_first_issue #bug
sent via relator
🚀 New issue to ag2ai/faststream by @carlodri
📝 LICENSE file is missing in source distribution (#2594)
Apache-2.0 requires publishing the LICENSE together with every distribution.
#good_first_issue #bug
sent via relator
📝 LICENSE file is missing in source distribution (#2594)
Apache-2.0 requires publishing the LICENSE together with every distribution.
#good_first_issue #bug
sent via relator
New issue to wemake-services/django-modern-rest by @sobolevn
Write tests for "double validation" problem (#85)
We need to be sure that double validation does not happen. Ever.
See what "double validation" is: fastapi/fastapi#3021
To be sure, we need to write tests for it.
Tests would be rather easy to write:
• One for
• One for
• One for raw data return
Also test disabled response validation mode to do 0 validations.
See the attached issue to learn how to make sure that double validation happened.
No matter what - we need to do a 1 or 0 validations.
#django_modern_rest #help_wanted #enhancement #good_first_issue
sent via relator
Write tests for "double validation" problem (#85)
We need to be sure that double validation does not happen. Ever.
See what "double validation" is: fastapi/fastapi#3021
To be sure, we need to write tests for it.
Tests would be rather easy to write:
• One for
@modify
• One for
@validate
• One for raw data return
Also test disabled response validation mode to do 0 validations.
See the attached issue to learn how to make sure that double validation happened.
No matter what - we need to do a 1 or 0 validations.
#django_modern_rest #help_wanted #enhancement #good_first_issue
sent via relator
🔥1
🚀 New issue to Lancetnik/FastDepends by @carlodri
📝 LICENSE file is missing in source distribution (#220)
#goodfirstissue
sent via relator
📝 LICENSE file is missing in source distribution (#220)
#goodfirstissue
sent via relator
😁5
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝
Similar to https://github.com/wemake-services/django-modern-rest/issues/93 we have several important public APIs for
But, we never test that they can actually be applied. This is a very simple issue with just 2 extra tests with existing examples from https://github.com/wemake-services/django-modern-rest/pull/95
#helpwanted #goodfirst_issue #enhancement
sent via relator
📝
Endpoint
customizations are not tested (#104)Similar to https://github.com/wemake-services/django-modern-rest/issues/93 we have several important public APIs for
Endpoint
customization: https://github.com/wemake-services/django-modern-rest/blob/9e1a24deb3a65683f991ebd14465b0b20e0c22b1/djangomodernrest/endpoint.py#L53-L58But, we never test that they can actually be applied. This is a very simple issue with just 2 extra tests with existing examples from https://github.com/wemake-services/django-modern-rest/pull/95
#helpwanted #goodfirst_issue #enhancement
sent via relator
👍2
🚀 New issue to wemake-services/django-modern-rest by @sobolevn
📝 Write a test for custom
Our endpoints have two methods to handle custom errors: https://github.com/wemake-services/django-modern-rest/blob/17edd60e1d13b88f4ee2c229095c237b1919b415/djangomodernrest/endpoint.py#L118-L132
But, they are not tested to be customized. Please, add two cases with:
1. Async endpoint and
2. Sync endpoint and
#helpwanted #goodfirst_issue #enhancement
sent via relator
📝 Write a test for custom
handle_error
and handle_async_error
types (#110)Our endpoints have two methods to handle custom errors: https://github.com/wemake-services/django-modern-rest/blob/17edd60e1d13b88f4ee2c229095c237b1919b415/djangomodernrest/endpoint.py#L118-L132
But, they are not tested to be customized. Please, add two cases with:
1. Async endpoint and
handle_async_error
that custom errors can be serialized to json the right way2. Sync endpoint and
handle_error
with the same logic#helpwanted #goodfirst_issue #enhancement
sent via relator