Commit ee6798e
authored
🌟 [Major]: Process-PSModule v5 (#205)
This pull request cleans up and consolidates the process into focussing
on a single workflow, the `workflow.yml` file. It also updates some more
logic and addresses some security issues for the checkout action.
## Details
- Removed the workflow `CI.yml` which has previously been used for
nightly runs, checking that the logic works.
- Mitigation: Adjusted `workflow.yml` to work for the same scenarios
replacing `CI.yml` for this usecase.
- Fixes #204
- Cleaned up some of the job flow depending on the different scenarios.
| Job | Open/Updated PR | Merged PR | Abandoned PR | Manual Run |
|-----|-----------------|-----------|--------------|------------|
| **Get-Settings** | ✅ Always | ✅ Always | ✅ Always | ✅ Always |
| **Lint-Repository** | ✅ Yes | ❌ No | ❌ No | ❌ No |
| **Build-Module** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Build-Docs** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Build-Site** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Test-SourceCode** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Lint-SourceCode** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Test-Module** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **BeforeAll-ModuleLocal** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Test-ModuleLocal** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **AfterAll-ModuleLocal** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Get-TestResults** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Get-CodeCoverage** | ✅ Yes | ✅ Yes | ❌ No | ✅ Yes |
| **Publish-Site** | ❌ No | ✅ Yes (only) | ❌ No | ❌ No |
| **Publish-Module** | ✅ Yes* | ✅ Yes* | ✅ Yes | ✅ Yes* |
- Only run linter on CI runs.
- Fast forward to "Publish-Module", which also handles removal of
prereleases (GitHub only).
- Only run "Publish-Site" on a merged PR.1 parent 5726f53 commit ee6798e
File tree
17 files changed
+421
-680
lines changed- .github
- prompts
- workflows
- .specify/memory
17 files changed
+421
-680
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
52 | 59 | | |
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
47 | 48 | | |
48 | 49 | | |
49 | 50 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
33 | 36 | | |
34 | 37 | | |
35 | 38 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
47 | 48 | | |
48 | 49 | | |
49 | 50 | | |
| |||
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
69 | 72 | | |
70 | 73 | | |
71 | 74 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
54 | 57 | | |
55 | 58 | | |
56 | 59 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
24 | 25 | | |
25 | 26 | | |
26 | 27 | | |
| |||
0 commit comments