Add sh linter; improve Makefile(s)#196
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sebrandon1 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Can we consider |
7b4a6bc to
69a66c1
Compare
69a66c1 to
d8eb871
Compare
0e8e064 to
2e0dd87
Compare
2e0dd87 to
05e8c9b
Compare
|
@sebrandon1: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Notable changes:
shfmt -wagainst the shell scripts in this repo so they pass the linter.Shell Formatting Enhancements:
.github/workflows/pre-main.yml: Added a new GitHub Actions workflow named "Shell Format Check" to validate shell script formatting usingshfmton pull requests and pushes to themainbranch.Makefile Updates:
telco-ran/configuration/Makefile: Introduced a newlinttarget to check shell script formatting withshfmt.telco-ran/configuration/extra-manifests-builder/Makefile: Added alinttarget for shell formatting checks and updated.PHONYto includelint. [1] [2]Code Style Improvements:
telco-ran/configuration/extra-manifests-builder/08-set-rcu-normal/test.sh: Adjusted indentation and spacing for better readability and consistency.telco-ran/configuration/Makefile: Simplified and standardized conditional checks in thecheckSourceCRsAnnotationtarget.These changes collectively aim to improve code quality, enforce consistent formatting, and streamline development workflows.