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

chore: refactor rigging pr decorator #30

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

GangGreenTemperTatum
Copy link
Collaborator

@GangGreenTemperTatum GangGreenTemperTatum commented Jan 24, 2025

AI-Generated Summary

  • Refactored the rigging_pr_decorator.py script to streamline PR description generation.
  • Replaced class-based structure with a more functional approach:
    • Removed the PRDiffData and PRDecorator classes.
    • Introduced a simplified generate_pr_description function.
  • Implemented a new get_diff function to handle git diff retrieval.
  • Modified main function to use typer for command line interfacing.
  • Adjusted the GitHub Actions workflow:
    • Changed PR trigger types from edited and reopened to opened and synchronize.
    • Removed unnecessary steps for direct diff retrieval, utilizing the refactored script.
  • Environment setup now specifies Python 3.10.
  • Improved error logging and streamlined dependency installation for better CI performance.
  • Overall improvements should lead to more efficient and maintainable code for PR description generation.

This summary was generated with ❤️ by rigging

@GangGreenTemperTatum GangGreenTemperTatum requested a review from a team as a code owner January 24, 2025 22:24
@GangGreenTemperTatum GangGreenTemperTatum merged commit 531f547 into main Jan 30, 2025
1 check passed
@GangGreenTemperTatum GangGreenTemperTatum deleted the ads/refactor-rigging-pr-decorator branch January 30, 2025 14:25
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