|
1 | 1 | # Description |
2 | 2 |
|
3 | | -Simplifies and consolidates the coverage report generation logic in the CI workflow, reducing redundant steps and dependencies. |
| 3 | +PR Description |
4 | 4 |
|
5 | 5 | ## Type of change |
6 | 6 |
|
7 | | -- [x] New feature (non-breaking change which adds functionality) |
8 | | -- [x] Infra/Build change (changes to CI/CD workflows or build scripts) |
| 7 | +- [ ] New feature (non-breaking change which adds functionality) |
| 8 | +- [ ] Infra/Build change (changes to CI/CD workflows or build scripts) |
9 | 9 | - [ ] Code refactoring |
10 | 10 | - [ ] Documentation change |
11 | 11 | - [ ] Bug fix |
12 | 12 | - [ ] Breaking change |
13 | 13 |
|
14 | 14 | ## Changes |
15 | 15 |
|
16 | | -- Merged `Generate Coverage Report` into the `Execute Tests` step — coverage `combine` and `json` generation now run inline after `bash test.sh`, following the same pattern as Megatron-LM-FL |
17 | | -- Coverage collection is gated on `test_type == 'unittest'` to avoid running for lint/debug groups, and `pip install` is done only once |
18 | | -- Removed `fetch-depth: 0` from checkout steps (not required for unit test runs) |
19 | | -- Removed unused/leftover scripts from the repository |
| 16 | +- Content 1 |
| 17 | +- Content 2 |
| 18 | +- Content 3 |
| 19 | +- Content 4 |
20 | 20 |
|
21 | 21 | ## Checklist |
22 | 22 |
|
23 | | -- [x] I have read and followed the contributing guidelines |
24 | | -- [x] The functionality is complete |
25 | | -- [x] I have commented my code, particularly in coverage report uploading steps |
26 | | -- [x] I have made corresponding changes to the documentation |
27 | | -- [x] My changes generate no new warnings |
28 | | -- [x] I have added/updated tests that prove my feature works on Cuda and Metax platform |
29 | | -- [x] New and existing unit tests pass locally on Cuda and Metax platform |
| 23 | +- [ ] I have read and followed the contributing guidelines |
| 24 | +- [ ] The functionality is complete |
| 25 | +- [ ] I have commented my code, particularly in coverage report uploading steps |
| 26 | +- [ ] I have made corresponding changes to the documentation |
| 27 | +- [ ] My changes generate no new warnings |
| 28 | +- [ ] I have added/updated tests that prove my feature works |
| 29 | +- [ ] New and existing unit tests pass locally |
0 commit comments