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

Run shellcheck from the project root #8

Merged
merged 3 commits into from
Jan 18, 2024
Merged

Conversation

samdoran
Copy link
Contributor

This allows project specific shellcheck settings to be applied.

It took me quite a while to figure out why this plugin wasn't using the settings in the .shellcheckrc at the root of the repository. It's because shellcheck walks up the directory path until it finds the first .shellcheckrc. The default directory where the shellcheck process is executed is ~/Library/Application Support/Nova/Extensions/[extension directory].

samdoran and others added 2 commits December 13, 2023 17:26
This allows project specific shellcheck settings to be applied.
@olly
Copy link
Owner

olly commented Jan 18, 2024

@samdoran thanks for the pull request. Apologies for the delay in getting back to you. I've changed your modifications slightly to keep the existing behaviour (where it runs with the working directory in the Application Support folder) when there's no workspace path configured.

I'll endeavour to get this built and released ASAP.

@olly olly merged commit 860323d into olly:main Jan 18, 2024
2 checks passed
@olly
Copy link
Owner

olly commented Jan 18, 2024

I've released v1.2 which includes this update. Thanks again for your contribution. 🙏

@olly olly mentioned this pull request Jan 18, 2024
@samdoran samdoran deleted the run-in-project-root branch January 18, 2024 20:27
@samdoran
Copy link
Contributor Author

Thank you so much!

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