Add Instruction to ensure Copyright header is present at start of script and module files by anamnavi · Pull Request #27408 · PowerShell/PowerShell
PR Summary
Add Instruction to ensure Copyright header below is present at start of script and module files
# Copyright (c) Microsoft Corporation. # Licensed under the MIT License.
PR Context
PR Checklist
- PR has a meaningful title
- Use the present tense and imperative mood when describing your changes
- Summarized changes
- Make sure all
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright header - This PR is ready to merge. If this PR is a work in progress, please open this as a Draft Pull Request and mark it as Ready to Review when it is ready to merge.
- Breaking changes
- None
- OR
- Experimental feature(s) needed
- Experimental feature name(s):
- User-facing changes
- Not Applicable
- OR
- Documentation needed
- Issue filed:
- Testing - New and feature
- N/A or can only be tested interactively
- OR
- Make sure you've added a new test if existing tests do not effectively test the code changed
Copilot AI review requested due to automatic review settings
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds a new GitHub Copilot instruction file to enforce PowerShell script/module file formatting, specifically ensuring a copyright/license header is present at the start of .ps1 and .psm1 files.
Changes:
- Introduces
.github/instructions/script-module-file-format.instructions.mdtargeting**/*.ps1and**/*.psm1. - Documents a required top-of-file copyright/license header and guidance to add/preserve it.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
TravisEz13
added
the
CL-Tools
label