Skip to content

Commit c6e4450

Browse files
authored
Merge pull request #41 from TechnologyEnhancedLearning/fix-quick-pr-run-now-in-master
Fix quick pr run now in master
2 parents 20bf2af + 314b519 commit c6e4450

File tree

6 files changed

+13
-74
lines changed

6 files changed

+13
-74
lines changed

.github/workflows/dev.yml

Lines changed: 4 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ jobs:
174174
175175
# - name: debug github commands
176176
# run: |
177-
# echo "but we want more complex versioning currently so wont use tag but will for release qqqq"
177+
# echo "but we want more complex versioning currently so wont use tag but will for release"
178178
# echo "as we will name commits like fix(broken-thing): changed broken line, and branches fix/broken-thing we should be able to use them in package names"
179179
# echo "branch tag $(git describe --tags --abbrev=0)"
180180
# echo "branch name $(git branch --show-current)"
@@ -518,16 +518,9 @@ jobs:
518518
# echo "BCLVersion: $BCL_VERSION"
519519
# echo "UseBCLProjectReference: $USE_BCL_PROJECT_REFERENCE"
520520

521-
# #qqqq not showing the value
522-
# - name: qqqq Display Directory.Packages.props
523-
# run: |
524-
# echo "if we can get this to say the version number then move it higher in the steps i think it may work"
525-
# echo "we dont want this <PackageVersion Include="TELBlazorComponentLibrary.GitPageBlazorWasm" Version="$(BCLVersion)" /> "
526-
# echo "try restore"
527-
# dotnet Restore
528-
# cat Directory.Packages.props
521+
529522

530-
# - name: qqqq is it because solution level not receive /p for project so need githun_env values
523+
# - name: is it because solution level not receive /p for project so need githun_env values
531524
# run: |
532525
# echo "BCLVersion=$BCL_VERSION" >> $GITHUB_ENV
533526
# echo "LocalPackagePath=$BCL_SOURCE" >> $GITHUB_ENV
@@ -536,7 +529,7 @@ jobs:
536529
# echo "UseBCLProjectReference=$USE_BCL_PROJECT_REFERENCE" >> $GITHUB_ENV
537530
# #echo "GITHUB_PACKAGES_TOKEN=$PACKAGES_TOKEN" >> $GITHUB_ENV
538531

539-
# - name: qqqq now try restore again
532+
# - name: now try restore again
540533
# run: |
541534
# dotnet Restore
542535
# cat Directory.Packages.props
@@ -545,39 +538,9 @@ jobs:
545538
# - name: Display Environment Variable (for debugging)
546539
# run: echo "BCLVERSION is $BCLVERSION"
547540

548-
# # - name: qqqqqqq here now try build
549-
# # run: |
550-
# # -e
551-
# # dotnet build
552-
# # cat Directory.Packages.props
553-
# # echo "doesnt work"
554-
# # +e
555-
556541

557542

558-
559-
560-
# - name: Publish client using client release .csproj logic
561-
# run: |
562-
# dotnet publish ./TestHostPrerenderWASM/TestHostPrerenderWASM.Client/TestHostPrerenderWASM.Client.csproj --configuration Release \
563-
# /p:IsLocalDev=$IS_LOCAL_DEV \
564-
# /p:BCLVersion=$BCL_VERSION \
565-
# /p:UseBCLProjectReference=$USE_BCL_PROJECT_REFERENCE
566-
# /p:GhPageRelease=true
567543

568-
569-
# #include hidden so we get jekyll
570-
# - name: Creating artifact of publish folder for test gh page workflow
571-
# uses: actions/upload-artifact@v4
572-
# with:
573-
# name: gh-blazor-components-wiki-site-using-package-${{env.BCL_VERSION}}
574-
# include-hidden-files: true
575-
# path: |
576-
# ./docs
577-
# ./docs/.nojekyll # explicitly include the .nojekyll file
578-
# retention-days: 90
579-
580-
581544
# - name: Debug artifact location
582545
# run: |
583546
# # Get the artifact list for the current workflow run

.github/workflows/pull_request.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ jobs:
66
branch-name-check:
77
name: Enforce Branch Name Convention
88
runs-on: ubuntu-latest
9-
if: success() || failure()
9+
#if: success() || failure()
1010
steps:
1111
- name: Validate Branch Name
1212
run: |
@@ -28,7 +28,7 @@ jobs:
2828
name: commitlint
2929
# qqqq repo will need branch protection rules and PR Template
3030
runs-on: ubuntu-latest
31-
if: success() || failure()
31+
#if: success() || failure()
3232
steps:
3333
- uses: actions/checkout@v4
3434
with:
@@ -42,7 +42,7 @@ jobs:
4242
gitguardian-scan:
4343
name: gitguardian
4444
runs-on: ubuntu-latest
45-
if: success() || failure()
45+
#if: success() || failure()
4646
steps:
4747
- uses: actions/checkout@v3
4848

@@ -55,7 +55,7 @@ jobs:
5555
Unit-Tests:
5656
name: todo unit tests
5757
runs-on: ubuntu-latest
58-
if: success() || failure()
58+
#if: success() || failure()
5959
steps:
6060
- name: See other project pipeline for Tests
6161
run: echo "qqqq See other project pipeline for Tests this is a placeholder"
@@ -65,7 +65,7 @@ jobs:
6565
E2E-Tests:
6666
name: todo e2e tests
6767
runs-on: ubuntu-latest
68-
if: success() || failure()
68+
#if: success() || failure()
6969
steps:
7070
- name: See other project pipeline for Tests
7171
run: echo "qqqq See other project pipeline for Tests this is a placeholder"
@@ -74,7 +74,7 @@ jobs:
7474
Code-Coverage:
7575
name: code coverage
7676
runs-on: ubuntu-latest
77-
if: success() || failure()
77+
#if: success() || failure()
7878
steps:
7979
- name: Code Coverage
8080
run: echo "TODO Code Coverage"

CHANGELOG.md

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,6 @@
1-
## [9.1.2](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.1...v9.1.2) (2025-04-15)
1+
## [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)
22

33

44
### Bug Fixes
55

6-
* **go:** go ([4690ddd](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/4690ddd92bf69a712899c6a75ca0c945b4b73fc3))
7-
8-
## [9.1.2-fix-quick-pr-run-now-in-master.1](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.1...v9.1.2-fix-quick-pr-run-now-in-master.1) (2025-04-15)
9-
10-
11-
### Bug Fixes
12-
13-
* **go:** go ([4690ddd](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/4690ddd92bf69a712899c6a75ca0c945b4b73fc3))
14-
15-
## [9.1.1](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.0...v9.1.1) (2025-04-15)
16-
17-
18-
### Bug Fixes
19-
20-
* **pr check:** check ([ae4bbad](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/ae4bbadd17de402900431967692b1d2792023318))
21-
22-
## [9.1.1-fix-pr-release-yml.1](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/compare/v9.1.0...v9.1.1-fix-pr-release-yml.1) (2025-04-15)
23-
24-
25-
### Bug Fixes
26-
27-
* **pr check:** check ([ae4bbad](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/ae4bbadd17de402900431967692b1d2792023318))
6+
* **pull request visible check:** go ([d29ba68](https://github.com/TechnologyEnhancedLearning/GitPageBlazorWASM/commit/d29ba686f77de02052b1ee79c18a73e8d71a63ee))

Package.BlazorComponentLibrary/Package.BlazorComponentLibrary.csproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@
4141
<Content Remove="packages.lock.json" />
4242
<Content Remove="wwwroot\background.png" />
4343
</ItemGroup>
44-
<!--qqqq building in the solution wasnt an issue before-->
4544
<ItemGroup>
4645
<Compile Remove="..\CICDPackageLocation\**\*.cs" />
4746
<Content Remove="..\CICDPackageLocation\**\*" />

ReadMe.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22

33

44

5-
## qqqq unlike in nuget feed git package feed doesnt display this file so to see it you need to look in the actual nuget package
6-
75
# Local Development
86
- inc the version in local props
97
- save

SharedPages/wwwroot/css/app.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,5 +75,5 @@
7575
code {
7676
color: #c02d76;
7777
}
78-
/* RCL Styles qqqq remember bundled scoped css we may not use it but it is extremely normal to so we may in future*/
78+
/* RCL Styles remember bundled scoped css we may not use it but it is extremely normal to so we may in future*/
7979
@import '_content/Package.BlazorComponentLibrary/Package.BlazorComponentLibrary.bundle.scp.css';

0 commit comments

Comments
 (0)