Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bower points to a fork #85

Closed
Nyholm opened this issue Aug 14, 2014 · 9 comments
Closed

Bower points to a fork #85

Nyholm opened this issue Aug 14, 2014 · 9 comments

Comments

@Nyholm
Copy link

Nyholm commented Aug 14, 2014

Consider the following command and output.

$ bower search mailcheck
Search results:

    mailcheck git://github.com/niftylettuce/mailcheck.git

The bower package mailcheck points to a fork by @niftylettuce. That fork is way behind master and it misses tag 1.1.0.

I suggest to create a new mailcheck package and point it to this repository. And also to update the docs.

@niftylettuce
Copy link
Contributor

I've updated my fork, it should be accurate now.

@niftylettuce
Copy link
Contributor

Also started watching this repo again, since I'll get updates now and can publish as they come in

@Nyholm
Copy link
Author

Nyholm commented Aug 22, 2014

What is the point of the fork? Are you mirroring this repo just because of the bower package?

@niftylettuce
Copy link
Contributor

Yes
On Aug 22, 2014 5:40 AM, "Tobias Nyholm" notifications@github.com wrote:

What is the point of the fork? Are you mirroring this repo just because of
the bower package?


Reply to this email directly or view it on GitHub
#85 (comment).

@Nyholm
Copy link
Author

Nyholm commented Aug 23, 2014

I suggest that @niftylettuce unregister the bower package (see how)

curl -X DELETE "https://bower.herokuapp.com/packages/mailcheck?access_token=TOKEN"

And then register the package again without the use of your fork.

bower register mailcheck geit://github.com/mailcheck/mailcheck.git  

That will make more sense and you do not have to manually mirror this repository.

@niftylettuce
Copy link
Contributor

Doing this now, thanks @Nyholm, didn't know they had unregister now.

@Nyholm
Copy link
Author

Nyholm commented Aug 23, 2014

That's great. Thanks 👍

@niftylettuce
Copy link
Contributor

@Nyholm done, please close issue, thank you for help here

@Nyholm
Copy link
Author

Nyholm commented Aug 23, 2014

Thank you!

@Nyholm Nyholm closed this as completed Aug 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants