-
Notifications
You must be signed in to change notification settings - Fork 94
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
[Feature] Make declarations of function to be POSIX compliant #334
Comments
If we implement this we need to add also a flag to Amber to define the bash version to support |
@Mte90 the POSIX version is supported by Bash and SH. Migrating to the POSIX version will not affect the existing bash compatibility |
I was trying to understand where in the code write amber/src/modules/function/invocation.rs Line 37 in 042ca2a
|
@Mte90 it's right here: amber/src/modules/function/declaration.rs Line 257 in 2e34e15
|
Is your feature request related to a problem? Please describe.
Currently functions generated by Amber are only bash compliant. Let's use the POSIX syntax for declaring functions.
Describe the solution you'd like
Change from:
to:
Describe alternatives you've considered
N/A
Additional context
This problem is related to the milestone to support
sh
The text was updated successfully, but these errors were encountered: