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.

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

allow parametrizing the issue title #38

Merged
merged 25 commits into from
Jul 7, 2024

Conversation

keewis
Copy link
Contributor

@keewis keewis commented Jul 6, 2024

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 merged commit 123161b into xarray-contrib:main Jul 7, 2024
3 checks passed
@keewis keewis deleted the query-by-title branch July 7, 2024 13:57
@agriyakhetarpal
Copy link
Contributor

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uniquely identify existing issues by title before editing issue descriptions
2 participants