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

Fix HTML-AutoCloseTag plugin location #1002

Open
wants to merge 1 commit into
base: 3.0
Choose a base branch
from

Conversation

pataquets
Copy link

No longer hosted at amirh/HTML-AutoCloseTag

No longer hosted at amirh/HTML-AutoCloseTag
@CLAassistant
Copy link

CLAassistant commented Dec 30, 2016

CLA assistant check
All committers have signed the CLA.

@qinyuhang
Copy link

Hi, I saw 2 other fix in this GitHub issue, and all you three solution is different from each other... #552

@chrisnicola
Copy link
Contributor

Two of the PRs update the location but don't install it. The one I added removes it as people have reported it doesn't work well anymore. It would be great if a decision could be made.

@pataquets
Copy link
Author

pataquets commented Jan 20, 2017

IMHO, if the plugin is broken (#552), removing it makes sense. Should be confirmed first, I seem to recall it being working when testing spf13-vim some time ago.
Otherwise, if it only has moved to another URL, it should be fixed. Removing something that might be useful for some people, just because of hosting switch seems not OK to me (#999, #1000), unless it is broken.
However, being unmaintained does not matter as long as it works.
Disclaimer: I'm not using it.

@pataquets
Copy link
Author

Updated above comment and cross-linked.

@conrado
Copy link

conrado commented Jun 9, 2017

repositories on https://github.com/vim-scripts don't get curated (PR's looked at, etc) I would recommend using a location like https://github.com/heracek/HTML-AutoCloseTag

@conrado
Copy link

conrado commented Jun 9, 2017

This is a duplicate of #999

@acgreek
Copy link

acgreek commented Nov 4, 2017

some needs to merge this.

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

Successfully merging this pull request may close these issues.

None yet

6 participants