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.

Sign up for GitHub

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

Jump to bottom

[WIP] src/test_report.py: Include patchwork patch metadata into test report #297

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[WIP] src/test_report.py: Include patchwork patch metadata into test report #297

wants to merge 2 commits into from

Conversation

yurinnick
Copy link
Contributor

@yurinnick yurinnick commented Aug 1, 2023 •
edited by gctucker
Loading

  • Include patch title, hash and web_url in the report, when available
  • Pass args from __init__ into _setup() in Service class

Fixes kernelci/kernelci-api#307

@yurinnick
src/base: Fix missing arguments for _setup() method
9255b5f 
Signed-off-by: Nikolay Yurin <yurinnick@meta.com>
@yurinnick
src/test_report.py: Include patchwork metadata into reporting
96440ab 
Signed-off-by: Nikolay Yurin <yurinnick@meta.com>
@yurinnick yurinnick mentioned this pull request Aug 1, 2023
Open
@gctucker gctucker added the staging-skip Don't test automatically on staging.kernelci.org label Aug 12, 2023
@gctucker
Copy link
Contributor

As with #299, I think this will depend on the rest of the code to have settled into a functional implementation first. Adding staging-skip label for now as it's likely to break every time the other PRs get updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
staging-skip Don't test automatically on staging.kernelci.org
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[RFC] Patch building and Webhooks API
2 participants
@yurinnick @gctucker