[Init]: Boilerplate scaffold removed#12
[Init]: Boilerplate scaffold removed#12Marius Storhaug (MariusStorhaug) merged 4 commits intomainfrom
Conversation
Super-linter summary
Super-linter detected linting errors For more information, see the GitHub Actions workflow run Powered by Super-linter POWERSHELL |
Super-linter summary
Super-linter detected linting errors For more information, see the GitHub Actions workflow run Powered by Super-linter POWERSHELL |
Super-linter summary
Super-linter detected linting errors For more information, see the GitHub Actions workflow run Powered by Super-linter POWERSHELL |
Super-linter summary
Super-linter detected linting errors For more information, see the GitHub Actions workflow run Powered by Super-linter POWERSHELL |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
There was a problem hiding this comment.
Pull request overview
This PR strips the repository down from the PSModule scaffold/template artifacts so the Jwt module is ready for real JWT implementation work, while keeping a single New-Jwt placeholder plus a minimal test and updated documentation.
Changes:
- Removed scaffold/template classes, functions, variables, formats/types, init/scripts, examples, and manifest override file.
- Added a
New-Jwtplaceholder function and a single Pester test validating its “not yet implemented” warning. - Updated README and lowered CI code coverage target to 0% during the placeholder phase.
Reviewed changes
Copilot reviewed 31 out of 32 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/PSModuleTest.Tests.ps1 | Removed scaffold Pester tests targeting PSModuleTest demo functions. |
| tests/Jwt.Tests.ps1 | Added minimal Pester test asserting New-Jwt emits an “not yet implemented” warning. |
| src/variables/public/SolarSystems.ps1 | Removed scaffold public variable data. |
| src/variables/public/Planets.ps1 | Removed scaffold public variable data. |
| src/variables/public/Moons.ps1 | Removed scaffold public variable data. |
| src/variables/private/PrivateVariables.ps1 | Removed scaffold private variable data. |
| src/types/FileInfo.Types.ps1xml | Removed scaffold type extension. |
| src/types/DirectoryInfo.Types.ps1xml | Removed scaffold type extension. |
| src/scripts/loader.ps1 | Removed scaffold script loader. |
| src/README.md | Removed scaffold internal src README. |
| src/modules/OtherPSModule.psm1 | Removed scaffold nested module example. |
| src/manifest.psd1 | Removed optional scaffold manifest override file. |
| src/init/initializer.ps1 | Removed scaffold init script. |
| src/header.ps1 | Removed scaffold header script. |
| src/functions/public/Test-PSModuleTest.ps1 | Removed scaffold public function. |
| src/functions/public/Set-PSModuleTest.ps1 | Removed scaffold public function. |
| src/functions/public/New-PSModuleTest.ps1 | Removed scaffold public function and aliases. |
| src/functions/public/New-Jwt.ps1 | Added New-Jwt placeholder function that emits a warning. |
| src/functions/public/Get-PSModuleTest.ps1 | Removed scaffold public function. |
| src/functions/private/Set-InternalPSModule.ps1 | Removed scaffold private function. |
| src/functions/private/Get-InternalPSModule.ps1 | Removed scaffold private function. |
| src/formats/Mygciview.Format.ps1xml | Removed scaffold format view definition. |
| src/formats/CultureInfo.Format.ps1xml | Removed scaffold format view definition. |
| src/finally.ps1 | Removed scaffold “finally” script. |
| src/data/Settings.psd1 | Removed scaffold data file. |
| src/data/Config.psd1 | Removed scaffold data file. |
| src/classes/public/Book.ps1 | Removed scaffold demo classes/enums. |
| src/classes/private/SecretWriter.ps1 | Removed scaffold demo class. |
| README.md | Rewritten to describe the Jwt module and current placeholder status. |
| examples/General.ps1 | Removed scaffold example script. |
| .github/PSModule.yml | Lowered code coverage target to 0% for the placeholder stage. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| New-Jwt | ||
| ``` | ||
|
|
||
| Returns a placeholder string indicating that the function is not yet implemented. |
| [CmdletBinding(SupportsShouldProcess)] | ||
| param() | ||
|
|
||
| begin {} | ||
|
|
||
| process { | ||
| if ($PSCmdlet.ShouldProcess('Creating a new JWT')) { | ||
| Write-Warning 'New-Jwt is not yet implemented.' | ||
| } |
The module repository no longer contains placeholder template code from the PSModule framework scaffolding. Only the
New-Jwtplaceholder function remains, signaling that the module is ready for actual JWT implementation work.Changed: Repository content
All template/demo artifacts are removed — classes (
Book,SecretWriter), functions (Get-PSModuleTest,New-PSModuleTest,Set-PSModuleTest,Test-PSModuleTest), variables (Planets,Moons,SolarSystems), data files, format definitions, type extensions, init scripts, nested modules, script loaders, examples, and the manifest override file.What remains in
src/:The
New-Jwtfunction emits a warning that it is not yet implemented. This gives CI a valid function to build and test against while development proceeds in #3.Changed: Test file
The test file is renamed from
PSModuleTest.Tests.ps1toJwt.Tests.ps1with a single test verifying the placeholder warning output.Changed: README
Rewritten to describe the Jwt module — name, purpose, installation, and current development status.
Technical Details
.github/PSModule.ymllowered from 50% to 0% — the placeholder function has no meaningful logic to covermanifest.psd1is removed because it is optional in the PSModule framework (only needed to override generated values); will be recreated in Implement JWT creation and signing functionality #3 when module metadata is defined