Skip to content
This repository was archived by the owner on Dec 11, 2017. It is now read-only.

Remove moot version property from bower.json#321

Merged
digitalBush merged 1 commit intodigitalBush:masterfrom
kkirsche:patch-1
Jun 19, 2015
Merged

Remove moot version property from bower.json#321
digitalBush merged 1 commit intodigitalBush:masterfrom
kkirsche:patch-1

Conversation

@kkirsche
Copy link
Contributor

@jason16v
Copy link

Sent from Windows Mail

From: Kevin Kirsche
Sent: ‎Thursday‎, ‎June‎ ‎11‎, ‎2015 ‎7‎:‎53‎ ‎PM
To: digitalBush/jquery.maskedinput

Per bower/spec@a325da3

Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property

You can view, comment on, or merge this pull request online at:

#321

Commit Summary
Remove moot version property from bower.json

File Changes
M bower.json (1)

Patch Links:
https://github.com/digitalBush/jquery.maskedinput/pull/321.patch
https://github.com/digitalBush/jquery.maskedinput/pull/321.diff


Reply to this email directly or view it on GitHub.

@kkirsche
Copy link
Contributor Author

Blank response

@digitalBush
Copy link
Owner

Intended to be used in the future when Bower gets a real registry where you can publish actual packages, but for now just leave it out.

If they intend to use it, is there any harm in leaving it?

@kkirsche
Copy link
Contributor Author

As stated above: Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property

@digitalBush
Copy link
Owner

Alright, works for me.

digitalBush added a commit that referenced this pull request Jun 19, 2015
Remove moot `version` property from bower.json
@digitalBush digitalBush merged commit bb66bf9 into digitalBush:master Jun 19, 2015
@digitalBush
Copy link
Owner

Thanks for helping keep things tidy around here! :)

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.

3 participants