Skip to content

Refactor!: Convert to Python, capture Gerrit environment parameters - #20

Merged
tykeal merged 1 commit into
lfreleng-actions:mainfrom
modeseven-lfreleng-actions:gerrit-metadata
Jan 7, 2026
Merged

Refactor!: Convert to Python, capture Gerrit environment parameters#20
tykeal merged 1 commit into
lfreleng-actions:mainfrom
modeseven-lfreleng-actions:gerrit-metadata

Conversation

@ModeSevenIndustrialSolutions

@ModeSevenIndustrialSolutions ModeSevenIndustrialSolutions commented Dec 6, 2025

Copy link
Copy Markdown
Contributor
  • Convert to Python, capture Gerrit environment parameters
  • Implement using modern PEP standards, pyproject.toml, uv tooling
  • Dynamic versioning for Python build system
  • Extensive Python tests for all functionality
  • Replaces fragile shell code, new features were too complex
  • Run through multiple audit passes to identify coding errors
  • Rename generate_summary -> github_summary

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 58 out of 60 changed files in this pull request and generated no new comments.


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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 57 changed files in this pull request and generated no new comments.


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

This comment was marked as outdated.

Comment thread src/extractors/changed_files.py Outdated
Comment thread src/utils/__init__.py Outdated
- Convert to Python, capture Gerrit environment parameters
- Implement using modern PEP standards, pyproject.toml, uv tooling
- Dynamic versioning for Python build system
- Extensive Python tests for all functionality
- Replaces fragile shell code, new features were too complex
- Run through multiple audit passes to identify coding errors
- Rename generate_summary -> github_summary
- Added uv.lock to repository

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Matthew Watkins <mwatkins@linuxfoundation.org>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 55 out of 56 changed files in this pull request and generated no new comments.


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

@tykeal
tykeal merged commit ceabcd9 into lfreleng-actions:main Jan 7, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Change potentially breaks upgrades refactor Refactoring of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants