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

Jump to bottom

TextMate integration? #3118

Closed
ScottFreeCode opened this issue Nov 25, 2017 · 1 comment · Fixed by lgaticaq/khaos-hubot-script#21, lgaticaq/hubot-bip#32, escaleno-ltda/hubot-toggl-payment#41, Drivetech/dmm-parser#51 or lgaticaq/bip#70
Closed

TextMate integration? #3118

ScottFreeCode opened this issue Nov 25, 2017 · 1 comment · Fixed by lgaticaq/khaos-hubot-script#21, lgaticaq/hubot-bip#32, escaleno-ltda/hubot-toggl-payment#41, Drivetech/dmm-parser#51 or lgaticaq/bip#70
Labels
area: repository tooling concerning ease of contribution good first issue new contributors should look here! type: chore generally involving deps, tooling, configuration, etc.

Comments

@ScottFreeCode
Copy link
Contributor

It appears Mocha's repo sports an unadvertised TextMate integration that can be installed using make tm.

Why is TextMate the only editor or IDE with an integration directly maintained as part of Mocha itself?

Can we move this out into its own repo? (Would that be semver-major? The install is with make instead of npm run and it's only available in the first place if you clone the repo, not if you npm i mocha.)

It doesn't seem like it's needed maintenance since I've been here, but it would be one fewer folders for new contributors to wonder about.
@ScottFreeCode ScottFreeCode added type: chore generally involving deps, tooling, configuration, etc. area: repository tooling concerning ease of contribution labels Nov 25, 2017
@boneskull boneskull added the good first issue new contributors should look here! label Dec 9, 2017
@Bamieh
Copy link
Contributor

Bamieh commented Dec 21, 2017 •
edited

@ScottFreeCode IMO Its not semver-major since its not bundled upon installation through npm. after all the versioning is set on npm's package.json file. it's like removing an unused file. Those who need it need to npm install another mocha module.

Bamieh added a commit that referenced this issue Jan 9, 2018
@Bamieh
Drop TextMate integration inside mocha closes #3118
9083129
@Bamieh Bamieh mentioned this issue Jan 9, 2018
Merged
boneskull pushed a commit that referenced this issue Jan 9, 2018
@Bamieh @boneskull
Drop TextMate integration inside mocha closes #3118
c7730a6
This was referenced Jan 18, 2018
Merged
Closed
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Closed
Merged
Closed
Merged
Closed
Merged
Merged
Merged
This was referenced Sep 23, 2018
Closed
Closed
Closed
Closed
Closed
This was referenced Oct 2, 2018
Closed
Closed
Closed
Open
This was referenced Oct 28, 2018
Merged
Closed
Closed
Merged
This was referenced Nov 4, 2018
Merged
Merged
Merged
This was referenced Nov 12, 2018
Closed
Merged
Closed
@renovate renovate bot mentioned this issue Nov 23, 2018
Merged
This was referenced Dec 1, 2018
Closed
Merged
Merged
This was referenced Dec 8, 2018
Merged
Closed
Merged
Closed
This was referenced Dec 23, 2018
Closed
Closed
sgilroy pushed a commit to TwineHealth/mocha that referenced this issue Feb 27, 2019
@Bamieh @boneskull
Drop TextMate integration inside mocha closes mochajs#3118
82f2461
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: repository tooling concerning ease of contribution good first issue new contributors should look here! type: chore generally involving deps, tooling, configuration, etc.
Projects
None yet
Milestone
No milestone
3 participants
@boneskull @Bamieh @ScottFreeCode