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

[Feature] Implement random.int function #759

Open
Cictrone opened this issue Apr 19, 2024 · 1 comment · May be fixed by #773
Open

[Feature] Implement random.int function #759

Cictrone opened this issue Apr 19, 2024 · 1 comment · May be fixed by #773
Assignees
@chase1k
Labels
eldritch Related to the Eldritch language feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Cictrone
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Eldritch currently can't create a random integer.

Describe the solution you'd like

Implement the random.int function that will return a random (i32) integer. This will theoretically be very similar to the random.bool function.

To see an example of what a PR that implements an eldritch function looks like, check out the PR for sys.get_user() here.
@Cictrone Cictrone added feature New feature or request help wanted Extra attention is needed good first issue Good for newcomers eldritch Related to the Eldritch language labels Apr 19, 2024
@Cictrone Cictrone assigned Cictrone and chase1k and unassigned Cictrone Apr 19, 2024
@Cictrone
Copy link
Collaborator Author

if you have questions lmk

@chase1k chase1k closed this as completed May 11, 2024
@chase1k chase1k reopened this May 11, 2024
@chase1k chase1k linked a pull request May 12, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chase1k chase1k

Labels
eldritch Related to the Eldritch language feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Random.int added spellshift/realm
2 participants
@Cictrone @chase1k