`toString` & `contains` dist API has something wrong

[adispring]

toString becomes toString$1
contains becomes contains$1

[buzzdecafe]

i wonder if this is related to #2327

[adispring]

In #2327, trim & keys spell typos are in source directory, but these two typos are in dist directory. This seems due to build:umd scripts or rollup which produces dist/ramda.js

[CrossEye]

ping @Andarist: Any ideas?

[Andarist]

Yeah, rollup sometimes renames identifiers like that to avoid conflicts between concatenated (scope hoisted) modules.

I had no idea that documentation is built directly from the umd build (which I guess is the case here). Gonna look into fixing this soon!

[Andarist]

So like I've stated above - rollup has renamed them due to name conflicts caused by this and this.

Imho trimming those suffixes should be done in the docs script to avoid this also in the future (alternative would be to rename those 2 mentioned functions, but I dont feel like its right way to solve this).

Need some time to figure out how this things is generated and then gonna send a PR with a fix.

[cht8687]

Hi guys, I've found the root cause for this and commited a fix for it.

#2334

[CrossEye]

Thanks, @cht8687, @adispring, and @Andarist!