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

Outboxes for 41 sort numbers as if they were strings #1

Closed
sniperrifle2004 opened this issue May 7, 2016 · 0 comments · Fixed by #3
Closed

Outboxes for 41 sort numbers as if they were strings #1

sniperrifle2004 opened this issue May 7, 2016 · 0 comments · Fixed by #3

Comments

@sniperrifle2004
Copy link
Contributor

This causes 10 to end up in the first position if it is in the sequence. This program has to change how it sorts the number strings (Better) or the inbox generator should only generate numbers between 1 and 9 (Which is rather hacky)
@sniperrifle2004 sniperrifle2004 mentioned this issue Dec 6, 2017
Closed
mrflip pushed a commit to mrflip/hrm-level-outbox-generator that referenced this issue Dec 6, 2017
Corrected tests in levels 4, 28, 41 (sorting errors)
6004f04 
Level 4 should swap entries, not sort them. The other levels need to handle sorts differently for ints and strings because Javascript derp.
Fixes atesgoral#1 and atesgoral/hrm-solutions#262
@mrflip mrflip mentioned this issue Dec 6, 2017
Merged
mrflip pushed a commit to mrflip/hrm-level-outbox-generator that referenced this issue Dec 6, 2017
Corrected tests in levels 4, 28, 41 (sorting errors)
00c0d9f 
Level 4 should swap entries, not sort them. The other levels need to handle sorts differently for ints and strings because Javascript derp.
Fixes atesgoral#1 and atesgoral/hrm-solutions#262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Corrected tests in levels 4, 28, 41 (sorting errors) mrflip/hrm-level-outbox-generator
1 participant
@sniperrifle2004