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

Add JS2Coffee for SublimeText 3 #6041

Merged
merged 1 commit into from
Jan 10, 2017
Merged

Add JS2Coffee for SublimeText 3 #6041

merged 1 commit into from
Jan 10, 2017

Conversation

voltechs
Copy link
Contributor

Please provide the following information:

Link to code repository: https://github.com/aziz/sublime-js2coffee
Link to the tags: https://github.com/aziz/sublime-js2coffee/tags

Also make sure you:

  1. Used "tags": true and not "branch": "master" (versioning docs)
  2. Ran the tests (tests docs)

@voltechs voltechs changed the title Add JS2Coffee for modern people Add JS2Coffee for SublimeText 3 Dec 29, 2016
@FichteFoll
Copy link
Collaborator

FichteFoll commented Jan 4, 2017

Considering that @nibblebot has abandoned that repo a long time ago and has not once commented on the back-referenced issue (nibblebot/sublime-js2coffee#14), I'm inclined to merge this immediately, but we do have a 14 day grace period in any case.

So, @nibblebot, unless you object within 14 days, your package will be replaced with @aziz's fork.

@aziz, please approve of this change as well. You also need to add a tag for the latest commit.

@aziz
Copy link
Contributor

aziz commented Jan 4, 2017

I approve this change. Though I'm not using this plugin on a daily basis anymore and if @nibblebot wants to merge my changes I'm happy to return it back to him.

@aziz
Copy link
Contributor

aziz commented Jan 4, 2017

Added 1.0.0 tag

@nibblebot
Copy link
Contributor

nibblebot commented Jan 4, 2017 via email

@FichteFoll FichteFoll merged commit 3670992 into wbond:master Jan 10, 2017
taylorzane pushed a commit to adelyte/package_control_channel that referenced this pull request Apr 7, 2017
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.

4 participants