ni / ni/nidaqmx-python

Get test result reporting working with PRs from forks

Open
#875 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

tech debt
Dominant language
Python
Stars
592
Forks
199
Avg merge
1d 16h
Merged PRs (30d)
10

Description

Tech Debt

Instead of listing each individual test configuration in the required checks, we should list the "Test Results" check. However, this doesn't work for PRs from forks of the repo.

To make this work, we need to change the way that we run EnricoMi/publish-unit-test-result-action:

https://github.com/EnricoMi/publish-unit-test-result-action?tab=readme-ov-file#support-fork-repositories-and-dependabot-branches

AB#3449935

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the GitHub Actions workflow configuration that runs EnricoMi/publish-unit-test-result-action, then read the linked documentation section on fork repositories and Dependabot branches. Update the action invocation so fork pull requests can report the consolidated Test Results check, and verify that the check works for those PRs.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, python
Domain
ci-cd
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.