Trigger tests for pull requests to all branches

sarthakpati commented 2 months ago

Fixes #ISSUE_NUMBER

Proposed Changes

ensure tests are triggered for pull requests initiated to all branches

Checklist

[x] CONTRIBUTING guide has been followed.
[x] PR is based on the current GaNDLF master .
[x] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[x] Function/class source code documentation added/updated (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
[x] Code has been blacked for style consistency and linting.
[x] If applicable, version information has been updated in GANDLF/version.py.
[x] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[x] Usage documentation has been updated, if appropriate.
[x] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[x] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].

github-actions[bot] commented 2 months ago

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

VukW commented 2 months ago

Hmm It seems to me that you are disabling these checks on any PR, aren't you? Say, this PR doesn't run checks anymore

VukW commented 2 months ago

GPT says triggers should look like

on:
  push:
    branches: [ master ]
  pull_request: {}

sarthakpati commented 2 months ago

pull_request: {}

Awesome, thanks!

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.41%. Comparing base (e674f47) to head (8efb957).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## new-apis_v0.1.0-dev #855 +/- ## ======================================================= + Coverage 94.40% 94.41% +0.01% ======================================================= Files 158 158 Lines 9345 9345 ======================================================= + Hits 8822 8823 +1 + Misses 523 522 -1 ``` | [Flag](https://app.codecov.io/gh/mlcommons/GaNDLF/pull/855/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mlcommons) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/mlcommons/GaNDLF/pull/855/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mlcommons) | `94.41% <ø> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mlcommons#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mlcommons / GaNDLF