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.

Sign up for GitHub

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

Jump to bottom

Change false, pwd, and true to substitutive built-ins #124

Merged
merged 2 commits into from
Mar 8, 2025
Merged

Change false, pwd, and true to substitutive built-ins #124

merged 2 commits into from
Mar 8, 2025

Conversation

c0d3-br3ak9r
Copy link
Contributor

  1. Replaced the definition of the commands from BI_MANDATORY to BI_SUBSTITUTIVE
  2. Updated the test cases to reflect the changes

Fixes #96

@magicant magicant self-requested a review March 4, 2025 13:08
@magicant magicant added this to the 2.59 milestone Mar 4, 2025
magicant
magicant approved these changes Mar 4, 2025
View reviewed changes
Copy link
Owner

@magicant magicant left a comment

Choose a reason for hiding this comment

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

I'll merge this later. Thank you!

@magicant magicant merged commit d6def52 into magicant:trunk Mar 8, 2025
2 checks passed
magicant added a commit that referenced this pull request Mar 8, 2025
@magicant
Document true, false, and pwd as substitutive built-ins
f036450 
This commit updates the documentation to reflect the change of the
type of the built-ins performed in:
#124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@magicant magicant magicant approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.59
Development

Successfully merging this pull request may close these issues.

Change false, pwd, and true to substitutive built-ins
2 participants
@c0d3-br3ak9r @magicant