Skip to content

allow parametrizing the issue title#38

Merged
keewis merged 25 commits intoscientific-python:mainfrom
keewis:query-by-title
Jul 7, 2024
Merged

allow parametrizing the issue title#38
keewis merged 25 commits intoscientific-python:mainfrom
keewis:query-by-title

Conversation

@keewis
Copy link
Copy Markdown
Contributor

@keewis keewis commented Jul 6, 2024 •
edited
Loading

This changes the query to use the search connection, which allows querying for issue titles. As a consequence, this allows parametrizing the issue title:

    strategy:
      matrix:
        os: ["ubuntu-latest", "macos-latest", "windows-latest"]
...
    steps:
...
      - name: generate the report
        if: failure()
        uses: keewis/issue-from-pytest-log@query-by-title
        with:
          log-path: pytest-log.jsonl
          issue-title: "CI: ${{matrix.os}} failed"

For examples, see keewis/reportlog-test#11, keewis/reportlog-test#12, and keewis/reportlog-test#13.

cc @andersy005

@keewis keewis mentioned this pull request Jul 6, 2024
Closed
@keewis keewis merged commit 123161b into scientific-python:main Jul 7, 2024
@keewis keewis deleted the query-by-title branch July 7, 2024 13:57
@agriyakhetarpal
Copy link
Copy Markdown
Member

agriyakhetarpal commented Jul 10, 2024

Thanks, @keewis! Using GraphQL looks better than my implementation. Thanks for tagging a release as well. Please give me a shout if you need help elsewhere, I am happy to do so – this is a fantastic project.

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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uniquely identify existing issues by title before editing issue descriptions

2 participants

@keewis @agriyakhetarpal