search

BinaryJazz / genrenator

Random genre creation
https://binaryjazz.us/genrenator-api/
GNU General Public License v3.0
11 stars 0 forks source link

5 remove spaces after genre #10

Closed jazzsequence closed 6 years ago

jazzsequence commented 6 years ago

I think this should solve the issue in #5 needs testing

jazzsequence commented 6 years ago

I feel like this is the more correct way of doing things but I'm still seeing issues. Rather than including spaces in the fragments, spaces should be added in texturize() conditionally. However there are cases where we're adding spaces to the end of something and that's causing problems when we're trying to contract that with the next thing (e.g. celtic coustica or c- folk or dream o, which should be celticcoustica, c-folk and dreamo, respectively).

I've bashed my head against this for long enough that I need to take a break. @binarygary maybe you want to take a look and see if you can figure out a solution?

jazzsequence commented 6 years ago

This is probably good to go now.