Skip to content

Two key changes to testsPython.yml for CI workflow#35

Open
junyan166-cpu wants to merge 1 commit intoFullStackWithLawrence:mainfrom
junyan166-cpu:main
Open

Two key changes to testsPython.yml for CI workflow#35
junyan166-cpu wants to merge 1 commit intoFullStackWithLawrence:mainfrom
junyan166-cpu:main

Conversation

@junyan166-cpu
Copy link
Copy Markdown

  1. if: always() on the notifications job — without this GitHub silently skips the job whenever python-unit-tests fails.
  2. Open GitHub Issue on failure step — uses actions/github-script@v7 with the built-in GITHUB_TOKEN (no new secrets needed). On failure it creates a labelled, assigned GitHub Issue with a direct link to the failed run.

Pull Request Template

Type of Change

  • New feature
  • Bug fix
  • Documentation
  • Refactor
  • Chore

Resolves

  • Fixes #[Add issue number here.]

Changes

Describe what this Pull Request does

Testing

Describe the testing that has been done or needs to be done

Screenshots

Add any relevant screenshots

Dependencies

List dependencies

Breaking Changes

Describe any breaking changes

1. if: always() on the notifications job — without this GitHub silently skips the job whenever python-unit-tests fails.
2. Open GitHub Issue on failure step — uses actions/github-script@v7 with the built-in GITHUB_TOKEN (no new secrets needed). On failure it creates a labelled, assigned GitHub Issue with a direct link to the failed run.
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.

1 participant