Skip to content
This repository was archived by the owner on Dec 11, 2025. It is now read-only.

Conversation

@TheWever
Copy link

@TheWever TheWever commented Aug 16, 2024

Summary

This PR ensures PEP 8 compliance across all cog files
Code readability was improved
Better naming schemes were introduced
Issues with existing commands were fixed

Refactored code to align with PEP 8 standards
Fixed typos
Renamed admintools.py to admin_tools.py
Renamed different sections more clearly
Renamed removetimeoutto remove_timeout
Renamed news to announce
admintools.py was renamed to admin_tools.py
Refactored code to align with PEP 8 standards
Fixed typos
Added type hints
Removed unused imports
Renamed Feedback.py to feedback.py
Refactored code
Added additional type hints
Renamed news command to announce
Renamed news.py cog to announcements.py
Refactored code to align with PEP 8 standards
Removed unused imports
Added type hints
Fixed typos
Renamed deathchat command to dead-chat (dead_chat internally)
Renamed deathchat.py cog to dead_chat.py
Fixed typos
Fixed command since it would not ping everyone but only the @everyone role of a specific server
Refactored code to align with PEP 8 standards
Added type hints
Removed unused imports
Refactored code
Added additional type hinting
Refactored code to align with PEP 8 standards
Added type hints
Removed unused imports
Fixed typos and rewrote some descriptions entirely
Renamed commands according to cogs/
@netlify
Copy link

netlify bot commented Aug 19, 2024

Deploy Preview for gregarious-strudel-6b15e5 failed.

Name Link
🔨 Latest commit 229e4d9
🔍 Latest deploy log https://app.netlify.com/sites/gregarious-strudel-6b15e5/deploys/66c34e18e5f0e00008da8376

Added versions to requirements
Removed unnecessary modules
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant