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

Shell variable interaction #276

Merged
merged 10 commits into from
Jul 5, 2024
Merged

Conversation

CymDeveloppement
Copy link
Member

Shell var interaction :
shell_constant_set
set constant in shell

shell_constant_get
get constant value

shell_var_set
set varible in shell

shell_var_get
get variable value

shell_isset
test if shell variable exist

shell_unset
unset varible in shell

the pr is related to #273

now shelle_var_set and shell_unset return true or false
@CymDeveloppement CymDeveloppement linked an issue Jul 5, 2024 that may be closed by this pull request
Copy link
Member

@Ph0enixKM Ph0enixKM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just use the nameof keyword appropriately and let me know if you agree with the failing of the functions that I suggested

CymDeveloppement and others added 7 commits July 5, 2024 10:36
Co-authored-by: Phoenix Himself <pkaras.it@gmail.com>
Co-authored-by: Phoenix Himself <pkaras.it@gmail.com>
Co-authored-by: Phoenix Himself <pkaras.it@gmail.com>
Co-authored-by: Phoenix Himself <pkaras.it@gmail.com>
Co-authored-by: Phoenix Himself <pkaras.it@gmail.com>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@Ph0enixKM Ph0enixKM merged commit 2b96f87 into amber-lang:master Jul 5, 2024
1 check passed
@Ph0enixKM
Copy link
Member

This does not work on macOS

Screenshot 2024-07-05 at 22 56 43

@CymDeveloppement CymDeveloppement deleted the shell_var branch July 7, 2024 08:37
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.

set global shell var in amber
3 participants