Skip to content

Conversation

@grigorescu
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 17, 2024

File Coverage
All files 81%
manage.py 56%
config/asgi.py 36%
config/consumers.py 78%
config/urls.py 82%
config/settings/base.py 74%
config/settings/local.py 75%
scram/contrib/sites/migrations/0003_set_site_domain_and_name.py 90%
scram/route_manager/authentication_backends.py 93%
scram/route_manager/models.py 70%
scram/route_manager/views.py 67%
scram/route_manager/api/serializers.py 72%
scram/route_manager/api/views.py 84%
scram/route_manager/tests/test_websockets.py 71%
scram/route_manager/tests/acceptance/steps/common.py 75%
scram/route_manager/tests/acceptance/steps/ip.py 70%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 85adfa8

@grigorescu grigorescu changed the title Add docstrings to all scram classes and functions and require them moving forward Add docstrings to all classes and functions and require them moving forward Nov 17, 2024
@grigorescu grigorescu enabled auto-merge November 17, 2024 22:00
Copy link
Collaborator

@crankynetman crankynetman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fantastic! LGTM!

@grigorescu grigorescu merged commit 25e1069 into develop Nov 18, 2024
8 checks passed
@grigorescu grigorescu deleted the topic/vlad/docstrings branch November 18, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants