Skip to content

Commit 3d5e6dc

Browse files
authored
Merge pull request #43 from TechnologyEnhancedLearning/fix-last-branch-check
Fix last branch check
2 parents 123c2dd + c536b89 commit 3d5e6dc

File tree

2 files changed

+36
-64
lines changed

2 files changed

+36
-64
lines changed

.github/workflows/pull_request.yml

Lines changed: 36 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ jobs:
66
branch-name-check:
77
name: Enforce Branch Name Convention
88
runs-on: ubuntu-latest
9-
#if: success() || failure()
109
steps:
1110
- name: Validate Branch Name
1211
run: |
@@ -22,77 +21,77 @@ jobs:
2221
echo " build-*, feat-*, fix-*, bugfix-*, hotfix-*, build-*, chore-*, ci-*, docs-*, perf-*, refactor-*, revert-*, style-*, test-*, sample-*, security-*, config-*, bugfix-*, hotfix-*"
2322
exit 1
2423
fi
25-
#continue-on-error: true
2624
25+
# qqqq repo will need branch protection rules and PR Template
2726
commitlint:
2827
name: commitlint
29-
# qqqq repo will need branch protection rules and PR Template
3028
runs-on: ubuntu-latest
31-
#if: success() || failure()
3229
steps:
3330
- uses: actions/checkout@v4
3431
with:
3532
fetch-depth: 0
3633
- uses: wagoid/commitlint-github-action@v5
3734
with:
3835
configFile: .commitlintrc.json
39-
#continue-on-error: true #we dont want to package etc if bad commits also pre-commit preferred
4036

4137

42-
gitguardian-scan:
43-
name: gitguardian
44-
runs-on: ubuntu-latest
45-
#if: success() || failure()
46-
steps:
47-
- uses: actions/checkout@v3
38+
# gitguardian-scan:
39+
# name: gitguardian
40+
# runs-on: ubuntu-latest
41+
# #if: success() || failure()
42+
# steps:
43+
# - uses: actions/checkout@v3
4844

49-
- name: GitGuardian Scan
50-
uses: GitGuardian/gg-shield-action@v1
51-
env:
52-
GITGUARDIAN_API_KEY: ${{ secrets.GITGUARDIAN_API_KEY }}
53-
#continue-on-error: true # we dont want secrets going further and the keys need to be changed if this is triggered
45+
# - name: GitGuardian Scan
46+
# uses: GitGuardian/gg-shield-action@v1
47+
# env:
48+
# GITGUARDIAN_API_KEY: ${{ secrets.GITGUARDIAN_API_KEY }}
49+
# #continue-on-error: true # we dont want secrets going further and the keys need to be changed if this is triggered
5450

5551
Unit-Tests:
5652
name: todo unit tests
5753
runs-on: ubuntu-latest
58-
#if: success() || failure()
5954
steps:
6055
- name: See other project pipeline for Tests
6156
run: echo "qqqq See other project pipeline for Tests this is a placeholder"
62-
#continue-on-error: true
6357

6458

6559
E2E-Tests:
6660
name: todo e2e tests
6761
runs-on: ubuntu-latest
68-
#if: success() || failure()
6962
steps:
7063
- name: See other project pipeline for Tests
7164
run: echo "qqqq See other project pipeline for Tests this is a placeholder"
72-
#continue-on-error: true
65+
7366

7467
Code-Coverage:
7568
name: code coverage
7669
runs-on: ubuntu-latest
77-
#if: success() || failure()
7870
steps:
7971
- name: Code Coverage
8072
run: echo "TODO Code Coverage"
81-
#continue-on-error: true
8273

83-
All-Checks-Passed:
84-
name: all checks passed
85-
runs-on: ubuntu-latest
86-
needs:
87-
- branch-name-check
88-
- commitlint
89-
- gitguardian-scan
90-
- Unit-Tests
91-
- E2E-Tests
92-
- Code-Coverage
93-
if: ${{ success() }}
94-
steps:
95-
- name: all checks passed steps
96-
run: echo "allow all checks to run incase multiple fails then fail here"
74+
# All-Checks-Passed:
75+
# name: all checks passed
76+
# runs-on: ubuntu-latest
77+
# needs:
78+
# - branch-name-check
79+
# - commitlint
80+
# - gitguardian-scan
81+
# - Unit-Tests
82+
# - E2E-Tests
83+
# - Code-Coverage
84+
# #if: ${{ success() }}
85+
# steps:
86+
# - name: all checks passed steps
87+
# run: echo "allow all checks to run incase multiple fails then fail here"
9788

98-
89+
# wanted to do it like this
90+
# Code-Coverage:
91+
# name: code coverage
92+
# runs-on: ubuntu-latest
93+
#if: success() || failure()
94+
# steps:
95+
# - name: Code Coverage
96+
# run: echo "TODO Code Coverage"
97+
#continue-on-error: true

CHANGELOG.md

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +0,0 @@
1-
## [9.1.4](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.3...v9.1.4) (2025-04-15)
2-
3-
4-
### Bug Fixes
5-
6-
* **last:** check ([6c7a8c5](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/6c7a8c5afae46ad3aba910045a0650b3b2aaf8c7))
7-
8-
## [9.1.4-fix-last-branch-check.1](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.3...v9.1.4-fix-last-branch-check.1) (2025-04-15)
9-
10-
11-
### Bug Fixes
12-
13-
* **last:** check ([6c7a8c5](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/6c7a8c5afae46ad3aba910045a0650b3b2aaf8c7))
14-
15-
## [9.1.3](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.2...v9.1.3) (2025-04-15)
16-
17-
18-
### Bug Fixes
19-
20-
* **pull request visible check:** go ([d29ba68](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/d29ba686f77de02052b1ee79c18a73e8d71a63ee))
21-
22-
## [9.1.3-fix-quick-pr-run-now-in-master.1](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.2...v9.1.3-fix-quick-pr-run-now-in-master.1) (2025-04-15)
23-
24-
25-
### Bug Fixes
26-
27-
* **pull request visible check:** go ([d29ba68](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/d29ba686f77de02052b1ee79c18a73e8d71a63ee))

0 commit comments

Comments
 (0)