Refactor bash scripting for init.yaml #246
Labels
No Label
has
backport
in progress
invalid
kind
breaking
kind
bug
kind
build
kind
dependency
kind
deployment
kind
docs
kind
enhancement
kind
feature
kind
lint
kind
proposal
kind
question
kind
refactor
kind
security
kind
testing
kind
translation
kind
ui
need
backport
priority
critical
priority
low
priority
maybe
priority
medium
reviewed
duplicate
reviewed
invalid
reviewed
wontfix
skip-changelog
status
blocked
status
needs-feedback
status
needs-reviews
status
wip
upstream
gitea
upstream
other
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: gitea/helm-chart#246
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Non standard sh isn't really needed from what I can see in init.yaml
Could you explain a bit more detailed what you mean?
The scripts in https://gitea.com/gitea/helm-chart/src/branch/master/templates/gitea/init.yaml don't require any bashisms and could be refactored to standard sh
#!/bin/sh
I get what you mean. If you like, you can create a PR to change this. Any contribution is welcome.
I wonder though if this change will make it harder to contribute in the future.
Most people possibly write in
bash
and not insh
and if there's a strong limitation to usesh
only, people might eventually refrain from contributing?I don't see why? Bash isn't ubiquitous, take for example using any Alpine Linux images. There is no real limitation of Bash vs POSIX shell besides Bash making some things easier. Also the pervalnce of using
#!/bin/bash
needs to stop as a lot of the time it's just cargo culted.Please see my comment on PR 273. (#273 (comment))
Closing for now. Feel free to reopen.