Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Releasing Shell Linter v0.6.0 #40

Merged
merged 20 commits into from
Nov 11, 2021
Merged

Releasing Shell Linter v0.6.0 #40

merged 20 commits into from
Nov 11, 2021

Conversation

Azbagheri
Copy link
Owner

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

<ADD-DESCRIPTION-HERE>

Motivation and Context

<ADD-MOTIVATION-HERE>

Testing Steps

<ADD-STEPS-HERE>

Risks

<ADD-RISKS-HERE>

Additional Information

<ADD-ADDITIONAL-INFORMATION-HERE>

Azbagheri added 19 commits June 8, 2021 17:18
@Azbagheri
Merge pull request #33 from azohra/release/0.5.0
6375196 
Release/0.5.0
@Azbagheri
Adding support for Allowing 'source' outside of FILES
9d6e9cd
@Azbagheri
Added a test for external sourced files
f5b2114
@Azbagheri
Moved the external_sources test file to the test_dir directory
7979e89
@Azbagheri
Merge pull request #36 from azohra/feature/support-external-sources
16a423b 
Allow Shellcheck to follow sourced files (closes #28)
@Azbagheri
Renamed the script used to install shellcheck and bumped the shellche…
e300210 
…ck version.

Also upgraded the base image used in the Shell Linter's Dockerfile.
@Azbagheri
Modified the Dockerfile to install Shellcheck using our script
9df64fa
@Azbagheri
Fixed the Docker image build issue by removing sudo from the Shellche…
a556b1a 
…ck installation script
@Azbagheri
Updated the ci-workflow to rename the Shellcheck installation step an…
42c38b7 
…d use the right script
@Azbagheri
Changed the executable location in the install_shellcheck.sh script a…
d12a8e0 
…s it is not managed by the distribution and we won't need to use sudo in the script
@Azbagheri
Updated the install_shellcheck.sh script to run a cleaner installatio…
e498b4f 
…n by removing the shellcheck folder after the shellcheck executable is added to the path
@Azbagheri
Merge pull request #37 from azohra/feature/upgrade-shellcheck
721aa46 
Upgrade Shellcheck (fixes #34)
@Azbagheri
Added support for excluding files and folders from linting
d19a90c
@Azbagheri
Added tests for the path exclusion feature and improved some of the e…
765dd0e 
…xisting tests
@Azbagheri
Updated the README with the usage instructions about the path exclusi…
d61ceea 
…on feature
@Azbagheri
Modified the unscanned-files warning message to suggest using the 'ex…
3365d16 
…cluded_paths' input as a fix
@Azbagheri
Updated the README to mention the scripts that are scanned by default…
9470c31 
…. Also, set a default value for the excluded_paths parameter.
@Azbagheri
Changed the name of the input parameter for file exclusion to exclude…
ae0ae37 
…-paths
@Azbagheri
Merge pull request #38 from azohra/feature/exclude-paths-from-linting
780d6bb 
Exclude files and folders from linting (closes #35)
@Azbagheri Azbagheri requested a review from asadmansr November 10, 2021 03:30
@Azbagheri Azbagheri merged commit 6bbeaa8 into master Nov 11, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@asadmansr asadmansr asadmansr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Azbagheri @asadmansr