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

Deduplicate - acronym #112

Closed
3 tasks done
hobovsky opened this issue Aug 10, 2022 · 9 comments
Closed
3 tasks done

Deduplicate - acronym #112

hobovsky opened this issue Aug 10, 2022 · 9 comments
Assignees
Labels
discussion/deduplicate Discussion and vote what to do with duplicate kata

Comments

@hobovsky
Copy link
Contributor

hobovsky commented Aug 10, 2022

  1. makeAcronym
  • 7kyu
  • input with invalid types (really annoying), and invalid characters
  • 88% with 1.6k completions
  • 1 pending issue
  • 4 languages (Exclusive approved CoffeScript, Ruby)
  • Published Jun 2015, author active
  1. Make acronym
  • 7 kyu
  • valid inputs only
  • 88% with ~1k completions
  • 2 pending issues
  • 4 languages (2 unique) + 2 pending translations (both unique)
  • Published Aug 2016, author last active May 2022

Conclusion:

Filling gaps

  • Move CoffeeScript approved translation from 1. to 2.
  • Move Ruby approved translation from 1. to 2.
@hobovsky
Copy link
Contributor Author

Kata 1. has a very annoying requirement of input validation and throwing all kind of crap into user solution, so if this kata is decided to be kept, I'd vote to remove this requirement.
Random tests in kata 2. generate inputs expecting answers of only 1 character or 2 characters long, so maybe could be improved.

I am slightly inclined to keep 2. due to smaller effort required to move languages and no input validation.

@Blind4Basics
Copy link

retire 1.
Because transferring languages of 2 to one means opening issues in one about lack of random tests in those langauges

@ejini6969
Copy link

2 should be kept due to no input validation and translating to typed languages are less tedious !

@Kacarott
Copy link

retire 1

@EloiseRosen
Copy link

vote for retire 1

@akar-0
Copy link

akar-0 commented Aug 10, 2022

keep 2 only.

@hobovsky
Copy link
Contributor Author

From @KayleighWasTaken on Discord:

Not logged in to gh on phone but I vote we keep kata 2
Can fix the stuff with it's tests

@KayleighWasTaken
Copy link

From @KayleighWasTaken on Discord:

Not logged in to gh on phone but I vote we keep kata 2
Can fix the stuff with it's tests

Can confirm is me.

@wtlgo
Copy link

wtlgo commented Aug 11, 2022

Keep 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion/deduplicate Discussion and vote what to do with duplicate kata
Projects
Development

No branches or pull requests

9 participants