Add CI workflow #10
No reviewers
Labels
No labels
bug
compliance
dependencies
documentation
duplicate
enhancement
good first issue
help wanted
invalid
javascript
question
wontfix
No milestone
No project
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set.
Reference: adam/amazon-po-parser#10
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "feature/add-ci-workflow"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
Adds a CI workflow calling the org-level
ci-python-sam.yamlreusable workflow for Python linting. Applies ruff formatting and fixes toparse-mbox.py.Validation
ruff check .passes cleanruff format --check .passes cleanTests
No automated tests. CI workflow validated by syntax review.
PR Summary
Low Risk
Low risk: introduces a new PR-only CI workflow and the Python changes are formatting/cleanup (e.g., quoting, wrapping, removing unused import) with no intended logic changes.
Overview
Adds a pull-request CI workflow (
.github/workflows/ci.yaml) that delegates to the org reusableci-python-sam.yamlworkflow (lint/format), with SAM validation disabled.Reformats and lightly cleans up
parse-mbox.pyto satisfy ruff (consistent quoting/wrapping and removal of an unused import), with no functional changes intended.Reviewed by Cursor Bugbot for commit
702c65541a. Bugbot is set up for automated code reviews on this repo. Configure here.Reviewed: changes are limited to Black-style reformatting of
parse-mbox.py(no logic changes) and a new CI workflow that delegates to the sharedci-python-sam.yamlreusable workflow. Looks good — no issues to flag against the review criteria.