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.

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

shellcheck on stickler-ci doesn't understand the source directive #172

Closed
cailyoung opened this issue Jan 2, 2018 · 2 comments
Closed

shellcheck on stickler-ci doesn't understand the source directive #172

cailyoung opened this issue Jan 2, 2018 · 2 comments
Assignees
@markstory

Comments

@cailyoung
Copy link

cailyoung commented Jan 2, 2018 •
edited

To reproduce:

#! /bin/bash
# shellcheck source=scripts/inc/librelease.sh disable=SC1091
source "${SCRIPT_HOME}/inc/librelease.sh"

Live example:
CommandPost/CommandPost#929 (review)

When pasted in shellcheck.net no issues are found.

Perhaps it is running an older build of shellcheck?
@cailyoung cailyoung mentioned this issue Jan 2, 2018
Merged
@markstory
Copy link
Owner

I'll look into getting shellcheck updated. The installed version may be a few months old now.

@markstory markstory self-assigned this Jan 2, 2018
@markstory
Copy link
Owner

Shellcheck has been updated to 0.4.7 now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markstory markstory

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@markstory @cailyoung