Skip to content

Exclude specific Shellcheck issues during the scan #63

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.

Sign up for GitHub

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

Jump to bottom
Merged
Azbagheri merged 3 commits into from
Feb 5, 2025
Merged

Exclude specific Shellcheck issues during the scan #63

Azbagheri merged 3 commits into from
Feb 5, 2025

Conversation

@Azbagheri
Copy link
Owner

@Azbagheri Azbagheri commented Feb 5, 2025 •
edited
Loading

Pull Request Checklist

  • I have created an issue prior to creating this pull request
  • I have provided a detailed description and motivation regarding the change below
  • I have updated the test suite and documentation to support my change

Corresponding Issue

<ADD-ISSUE-HERE>

Description of Change

This features allows the shell-linter github action to exclude specific issues when running the ShellCheck linter. This is useful for accepting lint issues that are not required to be fixed at the moment or at all depending on the requirement.

Motivation and Context

Enhances the feature set of Shell-Linter and also a feature request by a user.

Testing Steps

More tests have been added to support the development

Risks

N/A

Additional Information

Updated the README on how users can apply these changes to their projects.

asadmansr and others added 3 commits March 11, 2022 11:27
@Azbagheri Azbagheri changed the title Feature/shellcheck exclude issues Exclude specific Shellcheck issues during the scan Feb 5, 2025
@Azbagheri Azbagheri linked an issue Feb 5, 2025 that may be closed by this pull request
Allow to exclude types of warnings #22
Closed
@Azbagheri Azbagheri merged commit 78e2040 into develop Feb 5, 2025
1 check passed
@Azbagheri Azbagheri mentioned this pull request Feb 5, 2025
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow to exclude types of warnings

2 participants

@Azbagheri @asadmansr