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

Add BASH Linters for CICD Code #18

Open
kj4ezj opened this issue Dec 22, 2022 · 0 comments
Open

Add BASH Linters for CICD Code #18

kj4ezj opened this issue Dec 22, 2022 · 0 comments
Assignees
Labels
CICD Continuous integration, continuous deployment - build and test system

Comments

@kj4ezj
Copy link
Contributor

kj4ezj commented Dec 22, 2022

Following devrel issue 135, this issue has a BASH linter being added to this repo, then being set as a required check on all CICD builds. As of this writing, the only BASH script are for CICD itself but those should be checked along with any other BASH scripts added to this repo.

Automation recommends implementing both ShellCheck and bashate with the following configurations:

shellcheck -x -f gcc -S error $FILE
bashate -i E006 $FILE

The shfmt utility can be used to automatically correct some types of linter errors, similar to yarn lint --fix with eslint in JavaScript. However, it should not be used for a CICD check.

cat $FILE | shfmt -i 4 -sr -d

See Also

  • devrel issue 80 - Standup DevHub Infrastructure
  • devrel issue 96 - DNS Mapping for learn.eosnetwork.com
  • devrel issue 134 - Update DevHub Site
  • devrel issue 135 - DevHub Frontend CICD
  • devrel issue 137 - Encrypt DevHub Backend
  • devhub issue 12 - Put AWS Resources into Terraform with a CICD Deployment
  • devhub issue 13 - Document CICD System
  • devhub issue 14 - Staging Environment
  • devhub issue 15 - Backend CICD
  • devhub issue 16 - Structures CICD
  • devhub issue 17 - Add a JavaScript/TypeScript Linter to the DevHub Projects
  • devhub issue 18 - Add BASH Linters for CICD Code
  • devhub issue 19 - Include Test Suite in CICD
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD Continuous integration, continuous deployment - build and test system
Projects
Status: Todo
Development

No branches or pull requests

1 participant