Skip to content

Reformating#39

Merged
ondratu merged 5 commits into
masterfrom
reformating
Jan 7, 2026
Merged

Reformating#39
ondratu merged 5 commits into
masterfrom
reformating

Conversation

@ondratu
Copy link
Copy Markdown
Contributor

@ondratu ondratu commented Jan 7, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR performs comprehensive code reformatting across the PoorWSGI project, standardizing code style and fixing several type hint issues. The changes include converting single quotes to double quotes throughout the codebase, applying consistent formatting (likely using Black/Ruff), and significantly expanding project documentation.

Key Changes:

  • Applied consistent code formatting with double quotes and improved line breaks throughout poorwsgi/request.py and related files
  • Fixed type hints for secret_key, path_args, and args setters to correctly reflect accepted types
  • Corrected license_files in setup.py from string to list format
  • Completely rewrote and expanded CONTRIBUTION.rst with comprehensive contribution guidelines including TDD approach and pre-commit setup
  • Added new ARCHITECTURE.rst file documenting the internal framework architecture
  • Removed .travis.yml (likely transitioning to a different CI system)

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
setup.py Changed path concatenation to path.join() and fixed license_files to use list format
poorwsgi/wsgi.py Fixed secret_key setter type hint to accept Union[str, bytes]
poorwsgi/request.py Reformatted quotes to double quotes, improved line breaks, fixed type hints for path_args and args setters
CONTRIBUTION.rst Complete rewrite with comprehensive contribution guidelines, TDD approach, and development setup instructions
ARCHITECTURE.rst New file documenting framework architecture, core objects, and request-response lifecycle
.travis.yml Removed Travis CI configuration file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread poorwsgi/request.py Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ondratu ondratu merged commit 5335043 into master Jan 7, 2026
5 checks passed
@ondratu ondratu deleted the reformating branch January 7, 2026 12:50
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.

2 participants