Skip to content
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

Update dependency ShellCheck to v0.9.0 - autoclosed #158

Closed
wants to merge 1 commit into from

Conversation

CyberShadow-Renovate
Copy link
Collaborator

@CyberShadow-Renovate CyberShadow-Renovate commented Jan 3, 2023

This PR contains the following updates:

Package Update Change
ShellCheck minor 0.8.0 -> 0.9.0

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Enabled.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

@CyberShadow
Copy link
Owner

With -x -a -P src aconfmgr:


In src/common.bash line 1708:
	while str=$(caller $frame)
                           ^----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
	while str=$(caller "$frame")


In src/helpers.bash line 350:
	file_content_filters[$pattern]=$function
                             ^------^ SC2004 (style): $/${} is unnecessary on arithmetic variables.

For more information:
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
  https://www.shellcheck.net/wiki/SC2004 -- $/${} is unnecessary on arithmeti...

Both of these look like false positive regressions :(

@CyberShadow-Renovate CyberShadow-Renovate changed the title Update dependency ShellCheck to v0.9.0 Update dependency ShellCheck to v0.9.0 - autoclosed Oct 29, 2023
@CyberShadow-Renovate CyberShadow-Renovate deleted the renovate/shellcheck-0.x branch October 29, 2023 13:46
@CyberShadow
Copy link
Owner

This got closed by accident as I was trying to update my Renovate fork. Continuation here: #177

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants