-
Notifications
You must be signed in to change notification settings - Fork 0
🪲[Fix]: Fix linting issue for main
#24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ual workflows; update module test description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes linting issues in the main branch by adding workflow configuration improvements and updating PowerShell module documentation. The changes enhance CI validation capabilities while making a minor documentation correction.
- Added environment variables to GitHub Actions workflow for enhanced code validation
- Updated PowerShell module function description to include "api"
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/Action-Test.yml | Added VALIDATE_ALL_CODEBASE and FILTER_REGEX_INCLUDE environment variables for improved CI validation |
| tests/srcTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1 | Updated function description to include "api" |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
tests/srcTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1
Outdated
Show resolved
Hide resolved
…cture for documentation validation
…Action-Test workflow
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…ion-Test workflow
…umentation validation
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…mentation validation
…ion-Test workflow
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
tests/srcTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1
Outdated
Show resolved
Hide resolved
…and remove redundant get changes step
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…date DEFAULT_BRANCH to use repository's default branch
…roup names and remove FILTER_REGEX_INCLUDE environment variable
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…g changes and restore VALIDATE_ALL_CODEBASE environment variable
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…ng and remove FILTER_REGEX_INCLUDE environment variable
…nd adjust environment variables for linting
…rkflow and set DEFAULT_BRANCH to reference the current branch
…t' branch for staging changes
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…mmit step in Action-Test workflow
…nused environment variable
… for staging changes
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
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 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
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.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This pull request updates the
Action-TestGitHub Actions workflow to improve its automation and linting capabilities. The most important changes are grouped below.Workflow triggers:
mainbranch, in addition to the existing triggers (workflow_dispatch,pull_request, andschedule).Job steps and linting improvements:
PSModule/GitHub-Script@v1action to list files and commit documentation changes has been removed, simplifying the workflow.Lint documentationstep now usessuper-linter/slim@v8.1.0instead of a pinned commit from v8.2.0, and several new environment variables have been added to enable broader validation (VALIDATE_NATURAL_LANGUAGE,VALIDATE_ALL_CODEBASE,USE_FIND_ALGORITHM).