Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ordinary functions for now, add two more functions, update tests, repackage cli #7

Merged
merged 13 commits into from
Dec 19, 2024

Conversation

tnm
Copy link
Contributor

@tnm tnm commented Dec 19, 2024

For features and factoring:

  • Removed async from GitHub API methods
  • Updated CLI to use synchronous calls
  • Removed aiohttp dependency
  • Renamed and reorganized PR creation functions for clarity
  • Created consistent pattern across all PR creation methods:
    • create_pr_from_file (single file)
    • create_pr_from_content (single content)
    • create_pr_from_multiple_contents (multiple content strings)
    • create_pr_from_files (multiple files)

For docs:

  • Updated Quick Start section in README
  • Added detailed explanations for each PR creation method
  • Added use cases and examples
  • Improved common options documentation

Dependencies and tests updated accordingly.

Copy link
Contributor

@josh josh left a comment

Choose a reason for hiding this comment

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

Nice

pyproject.toml Outdated Show resolved Hide resolved
@tnm tnm merged commit 0185330 into main Dec 19, 2024
3 checks passed
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