search

RndPhrase / RndPhrase.js

Javascript module to auto generate secure passwords
2 stars 0 forks source link

Rename library to just 'rndphrase'? #4

Open rlindsgaard opened 10 years ago

rlindsgaard commented 10 years ago

When importing a package in the firefox sdk only small letters, hyphens and underscores are allowed. This might be a problem elsewehere. Currently I fix it with https://github.com/RndPhrase/RndPhrase-Firefox/blob/master/scripts/install-packages.sh which is kind of a hack.

Munter commented 10 years ago

Well, 'rndphrase' is already the name of the original browser extension, so I wouldn't try to cause confusion by clashing the name space. Especially since this repository will look more official than Johans, organization name and all...

I'd say not being able to import libraries based on their name is a bug in Firefox

rlindsgaard commented 10 years ago

You might be right. If this is only a problem in Firefox there is no reason to change it.

However, if it shows that other browsers have similar problems I'd argue that the name should be changed. IMO RndPhrase is still primarily targeted for browser plugins. In such case we could choose 'rndphrase-js' or similar in order to avoid name collision and confusion.

rlindsgaard commented 10 years ago

Don't put "js" or "node" in the name. It's assumed that it's js, since you're writing a package.json file - source https://www.npmjs.org/doc/files/package.json.html

because of the namespace clash i agree that rndphrase is a bad choice. I am thinking that perhaps we can name it "rndphrase-improved".