Skip to content

Fix HTML-AutoCloseTag plugin location#1002

Open
pataquets wants to merge 1 commit into
spf13:3.0from
pataquets:patch-1
Open

Fix HTML-AutoCloseTag plugin location#1002
pataquets wants to merge 1 commit into
spf13:3.0from
pataquets:patch-1

Conversation

@pataquets

Copy link
Copy Markdown

No longer hosted at amirh/HTML-AutoCloseTag

No longer hosted at amirh/HTML-AutoCloseTag
@CLAassistant

CLAassistant commented Dec 30, 2016

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@qinyuhang

Copy link
Copy Markdown

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

@chrisnicola

Copy link
Copy Markdown
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

pataquets commented Jan 20, 2017

Copy link
Copy Markdown
Author

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
Copy Markdown
Author

Updated above comment and cross-linked.

@conrado

conrado commented Jun 9, 2017

Copy link
Copy Markdown

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

conrado commented Jun 9, 2017

Copy link
Copy Markdown

This is a duplicate of #999

@acgreek

acgreek commented Nov 4, 2017

Copy link
Copy Markdown

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.

6 participants