Page MenuHomePhabricator
Differential D9943

Remove textmate resources
ClosedPublic
Actions

Authored by joshuaspence on Jul 16 2014, 2:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 1, 10:11 AM
Unknown Object (File)
Sun, Sep 1, 10:11 AM
Unknown Object (File)
Sun, Sep 1, 10:11 AM
Unknown Object (File)
Sun, Sep 1, 10:07 AM
Unknown Object (File)
Sun, Aug 25, 9:56 PM
Unknown Object (File)
Aug 6 2024, 9:07 AM
Unknown Object (File)
Aug 2 2024, 10:30 AM
Unknown Object (File)
Jul 31 2024, 1:08 PM
View All Files
Subscribers
chad
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHU7afef299b8d3: Remove textmate resources
Summary

I don't think that these are very useful anymore (although perhaps they are?)

Test Plan

N/A

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 23854.Jul 16 2014, 2:12 AM
joshuaspence retitled this revision from to Remove textmate resources.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 16 2014, 2:12 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1707: Diff 23854.Jul 16 2014, 2:13 AM
chad added a subscriber: chad.Jul 16 2014, 2:40 AM

epriestley uses textmate

epriestley accepted this revision.Jul 16 2014, 11:46 AM
epriestley edited edge metadata.

I use TextMate but haven't seen anyone else mentioning it or asking for help with these. They also aren't particularly Phabricator-related. There's an open task somewhere like "document how to configure every editor in the world to strip trailing whitespace when files are saved", and if we ever pursue that we could publish this kind of stuff as a separate repo somewhere.

This revision is now accepted and ready to land.Jul 16 2014, 11:46 AM
joshuaspence added a comment.Jul 16 2014, 12:34 PM
In D9943#8, @epriestley wrote:

They also aren't particularly Phabricator-related.

That was my main motivation for this diff

joshuaspence closed this revision.Jul 16 2014, 12:35 PM
joshuaspence updated this revision to Diff 23880.

Closed by commit rPHU7afef299b8d3 (authored by @joshuaspence).

chad added a comment.Jul 16 2014, 2:42 PM

:3

Revision Contents
Changeset List

Diff 23880

View Options

resources/textmate/local_definition.tmCommand

Loading...
View Options

resources/textmate/php_property.tmSnippet

Loading...
View Options

resources/textmate/trailing_whitespace.tmMacro

Loading...
Phabricator · Built by Phacility