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

[Feature] Create len builtin #441

Closed
2 of 5 tasks
Ph0enixKM opened this issue Sep 2, 2024 · 2 comments · Fixed by #545
Closed
2 of 5 tasks

[Feature] Create len builtin #441

Ph0enixKM opened this issue Sep 2, 2024 · 2 comments · Fixed by #545
Labels
enhancement New feature or request

Comments

@Ph0enixKM
Copy link
Member

Ph0enixKM commented Sep 2, 2024

Is your feature request related to a problem? Please describe.
There was a problem with len function - where to put it, because it fits well in std/text and std/array. The len function is so atomic that should be implemented as a built-in.

Describe the solution you'd like

Describe alternatives you've considered
Moving len to std/core. This would complicate things even more.

Additional context
N/A

@Ph0enixKM Ph0enixKM added the enhancement New feature or request label Sep 2, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Amber Project Sep 2, 2024
@Ph0enixKM Ph0enixKM added this to the Amber 0.4.0-alpha milestone Sep 2, 2024
@b1ek

This comment has been minimized.

@Mte90

This comment has been minimized.

@Ph0enixKM Ph0enixKM changed the title [Feature] Move len function from std/text to std/core [Feature] Create len builtin Sep 4, 2024
@Mte90 Mte90 removed the stdlib label Oct 2, 2024
Mte90 added a commit to Mte90/Amber that referenced this issue Oct 2, 2024
Mte90 added a commit to Mte90/Amber that referenced this issue Oct 2, 2024
@Ph0enixKM Ph0enixKM linked a pull request Oct 26, 2024 that will close this issue
Mte90 added a commit that referenced this issue Oct 28, 2024
Co-authored-by: Daniele Scasciafratte <[email protected]>
@github-project-automation github-project-automation bot moved this from 🆕 New to 🏁 Done in Amber Project Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants