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

Switch to megalinter #31

Merged
merged 3 commits into from Feb 27, 2022
Merged

Switch to megalinter #31

merged 3 commits into from Feb 27, 2022

Conversation

unixorn
Copy link
Owner

@unixorn unixorn commented Feb 27, 2022

Description

  • Switch to megalinter.
  • Upload megalinter results as GitHub artifacts

Checklist

  • All new and existing tests pass.
  • Any added/updated scripts added use #!/usr/bin/env interpreter instead of potentially platform-specific direct paths (#!/bin/sh and #!/bin/bash are allowed exceptions)
  • Added/updated scripts are marked executable
  • Scripts do not have a language file extension unless they are meant to be sourced and not run standalone. No end-user should have to know if a script was written in bash, python, ruby or whatever. Not including file extensions makes it easier to rewrite the script in another language later without having to change every reference to the previous version.
  • I have confirmed that the link(s) in my PR are valid.

License Acceptance

  • This repository is Apache version 2.0 licensed (some scripts may have alternate licensing inline in their code) and by making this PR, I am contributing my changes to the repository under the terms of the Apache 2 license.

- Switch to megalinter.
- Upload megalinter results as GitHub artifacts

Signed-off-by: Joe Block <jpb@unixorn.net>
Signed-off-by: Joe Block <jpb@unixorn.net>
@github-actions
Copy link

github-actions bot commented Feb 27, 2022

Mega-Linter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 1 0 0.02s
✅ COPYPASTE jscpd yes no 1.17s
✅ CREDENTIALS secretlint yes no 0.93s
✅ GIT git_diff yes no 0.01s
✅ MARKDOWN markdownlint 1 0 0.35s
✅ MARKDOWN markdown-link-check 1 0 1.54s
✅ MARKDOWN markdown-table-formatter 1 0 0.3s
❌ SPELL cspell 2 15 1.78s
✅ SPELL misspell 2 0 0.04s
✅ YAML prettier 1 0 1.41s
✅ YAML v8r 1 0 1.95s
✅ YAML yamllint 1 0 0.15s

See errors details in artifact Mega-Linter reports on GitHub Action page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

Signed-off-by: Joe Block <jpb@unixorn.net>
@unixorn unixorn force-pushed the switch-to-megalinter branch 5 times, most recently from f0442c5 to eca4cb3 Compare February 27, 2022 16:48
@unixorn unixorn merged commit 6ec5ad5 into main Feb 27, 2022
@unixorn unixorn deleted the switch-to-megalinter branch February 27, 2022 16:53
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.

None yet

1 participant