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

✨ add a fancy GH action for automatic shellcheck on every push to master and on every PR… #69

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

✨ add a fancy GH action for automatic shellcheck on every push to master and on every PR… #69

wants to merge 2 commits into from

Conversation

thomasmerz
Copy link
Contributor

You'll need to enable Actions in Settings of this repo.

@thomasmerz
Copy link
Contributor Author

@cleanbrowsing , have you seen my PR and can you please review and merge it? Thanks…

@thomasmerz
Copy link
Contributor Author

@cleanbrowsing
Here you can see how this GH action will look like before dnstest.sh has been shellcheck'ed.
And here you can see how GH action will look like after dnstest.shhas beenshellcheck`'ed.

This was referenced Mar 17, 2022
Open
Open
@thomasmerz
Copy link
Contributor Author

Ping @cleanbrowsing …

@thomasmerz
Copy link
Contributor Author

Ping @cleanbrowsing …

@cleanbrowsing
Copy link
Owner

Checking...

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.
2 participants
@thomasmerz @cleanbrowsing