Skip to content
This repository was archived by the owner on Feb 29, 2024. It is now read-only.

Support for Bower packaging#75

Closed
spotshare-nick wants to merge 1 commit intoKrinkle:masterfrom
spotshare-nick:master
Closed

Support for Bower packaging#75
spotshare-nick wants to merge 1 commit intoKrinkle:masterfrom
spotshare-nick:master

Conversation

@spotshare-nick
Copy link

Wiredep as well as other tools read the bower.json file for linking purposes. Created and tested valid bower.json file.

@Krinkle
Copy link
Owner

Krinkle commented Jul 3, 2017

@nblasgen Thanks for your proposal, but I'll have to decline at this time.

I do not support use of Bower. And the Bower maintainers themselves also don't recommend its use anymore. Going forward, you should use the npm registry (possibly through Yarn or Webpack).

While Bower is maintained, we recommend yarn and webpack for new front-end projects!

In a related note, given the purpose of this particular project (jquery-json), I would recommend thinking twice before using it. It's purpose is to provide JSON parsing capabilities to older browsers. Virtually every modern browser (Over 98%, depending on the demographics of your target audience) implements JSON.parse and JSON.stringify as part of the JavaScript run-time.

See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON

If you need support for older browsers, I'd recommend using json2.js, from https://github.com/douglascrockford/JSON-js.

@Krinkle Krinkle closed this Jul 3, 2017
@spotshare-nick
Copy link
Author

I fully agree, but the developers of Freeswitch have chosen to pick your library for their JSON handling. So while I could patch their code to use the JSON object (as any code that relies on new browser features should do), I found the process that much easier to just adjust your code to make the existing jquery-json bower package useable.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants