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

Add functions to list library #36

Merged
merged 9 commits into from
Aug 23, 2024
Merged

Conversation

V-Ask
Copy link

@V-Ask V-Ask commented Apr 16, 2024

Added following functions to list library:
contains
filter
first
slice
insert_at_index

PeaceMonster and others added 4 commits April 16, 2024 12:05
Added following functions to list library:
contains
filter
first
slice
insert_at_index
Copy link
Contributor

Choose a reason for hiding this comment

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

should not be in the repo

Copy link
Contributor

Choose a reason for hiding this comment

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

this probably need no modifications; maybe move this to another place?

Copy link
Contributor

Choose a reason for hiding this comment

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

contains seems to be doing the same as elem that's already in the list

Copy link
Contributor

@aslanix aslanix left a comment

Choose a reason for hiding this comment

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

see inline comments

Copy link
Author

@V-Ask V-Ask left a comment

Choose a reason for hiding this comment

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

Unnecessary files removed, and removed contains

@V-Ask V-Ask requested a review from aslanix June 3, 2024 09:52
@aslanix aslanix changed the base branch from dev to dev-raft-2024 August 23, 2024 08:23
@aslanix aslanix merged commit 44c944d into TroupeLang:dev-raft-2024 Aug 23, 2024
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.

3 participants