Skip to content

Generic webhook#1

Merged
myakove merged 2 commits intomainfrom
refactor-to-be-generic-webhook
Sep 6, 2022
Merged

Generic webhook#1
myakove merged 2 commits intomainfrom
refactor-to-be-generic-webhook

Conversation

@myakove
Copy link
Copy Markdown
Collaborator

@myakove myakove commented Sep 6, 2022

No description provided.

@myakove myakove self-assigned this Sep 6, 2022
@myakove myakove mentioned this pull request Sep 6, 2022
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Sep 6, 2022

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR.

Available user actions:

  • To mark PR as verified add !verified to a PR comment, to un-verify add !-verified to a PR comment.
    Verified label removed on each new commit push.
  • To cherry pick a merged PR add !cherry-pick <target branch to cherry-pick to> to a PR comment.

@myakove myakove merged commit ddc08df into main Sep 6, 2022
@myakove myakove deleted the refactor-to-be-generic-webhook branch September 6, 2022 06:28
myakove added a commit that referenced this pull request Mar 15, 2026
- Use config.data_dir/logs for JsonLogHandler directory (Qodo #1)
- Add encoding="utf-8" to JSONL file writes (Qodo #2)
- Thread-safe handler attachment with log_dir match check (CodeRabbit)
- Use _detect_success() in _derive_level() for all failure indicators (CodeRabbit)
- Filter log_entry type in get_workflow_steps_json() (CodeRabbit)
- Remove unused mock_datetime fixture params (CodeRabbit)
- Add regression tests for can_merge=False and success=False
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant