Skip to content

Fix commit-message label parsing on issue webhooks - #36

Open
michael-o wants to merge 1 commit into
ahebrank:masterfrom
michael-o:fix-commit-label-parsing
Open

michael-o wants to merge 1 commit into
ahebrank:masterfrom
michael-o:fix-commit-label-parsing

Conversation

@michael-o

Copy link
Copy Markdown
Contributor

Three bugs in the "parse commit message, manage labels" feature caused it to crash on every use: parse_commit_labels() was called without its required labels argument, simplify_labels() was called with the raw issue label_ops list instead of the full parse result and without board_labels (defaulting to None), and ops[1] was a str being passed to .decode('utf-8'), which str has no method for in Python 3.

Three bugs in the "parse commit message, manage labels" feature caused
it to crash on every use: parse_commit_labels() was called without its
required labels argument, simplify_labels() was called with the raw
issue label_ops list instead of the full parse result and without
board_labels (defaulting to None), and ops[1] was a str being passed
to .decode('utf-8'), which str has no method for in Python 3.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

1 participant