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

Integer statements #150

Merged
merged 1 commit into from
Jun 22, 2023
Merged

Integer statements #150

merged 1 commit into from
Jun 22, 2023

Conversation

dstallenberg
Copy link
Contributor

@dstallenberg dstallenberg commented Jun 22, 2023

Currently, we only have a statement for the JavaScript Numeric type, which encompasses integers and decimals. However, most problems will require integers and not doubles. Using the numeric type makes it much more challenging to satisfy certain branches.

This PR adds an integer type next to the default numeric type so that the search algorithm can figure out which type works better.

@dstallenberg dstallenberg changed the base branch from main to patch-branch-distance June 22, 2023 15:47
@dstallenberg dstallenberg changed the base branch from patch-branch-distance to main June 22, 2023 15:47
@mitchellolsthoorn mitchellolsthoorn marked this pull request as ready for review June 22, 2023 17:54
@mitchellolsthoorn mitchellolsthoorn requested a review from a team as a code owner June 22, 2023 17:54
@mitchellolsthoorn mitchellolsthoorn merged commit 3b6c075 into main Jun 22, 2023
@mitchellolsthoorn mitchellolsthoorn deleted the implement-integer-statements branch June 22, 2023 17:57
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.

2 participants