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

support leesei/logcat.tmLanguage #20

Closed
wants to merge 3 commits into from
Closed

support leesei/logcat.tmLanguage #20

wants to merge 3 commits into from

Conversation

leesei
Copy link
Contributor

@leesei leesei commented Apr 27, 2015

The original ADB was based on quarnster/ADBView.
I've written another syntax (with better scopes) at leesei/logcat.tmLanguage.
Also updated WARN's highlight.

The original ADB was based on [quarnster/ADBView](https://github.com/quarnster/ADBView).
I've written another syntax (with better scopes) at
[leesei/logcat.tmLanguage](https://github.com/leesei/logcat.tmLanguage).
Also updated WARN's highlight.
@MattDMo
Copy link
Owner

MattDMo commented Jun 30, 2015

Thanks for the PR, and I apologize for not checking it out sooner. Your additions look fine, but I'd still like to support ADBView for those who haven't changed to your version. Could you modify your changes so that both sets of scopes still work? For example, constant.other.adb.timestamp could become constant.other.adb.timestamp, string.logcat.timestamp, and the name of the rule just remain Timestamp.

Thanks!

@leesei
Copy link
Contributor Author

leesei commented Jul 1, 2015

I see. I will create a new PR that merges my several updates.

@leesei leesei closed this Jul 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants