Skip to content
This repository has been archived by the owner. It is now read-only.

Add Syntax Highlighting for SugarSS #13

Merged
merged 5 commits into from Oct 26, 2016
Merged

Conversation

@maxinspace
Copy link
Contributor

@maxinspace maxinspace commented Oct 25, 2016

Totally based on Atom plugin.

Note for Reviewers:

  1. About README:
    Should I simply add Link to SugarSS syntax to current PostCSS README?
@hudochenkov
Copy link
Owner

@hudochenkov hudochenkov commented Oct 25, 2016

  1. Yeah, just add something to current readme about SugarSS, so people would know that this syntax highlighting works with SugarSS.
  2. Don't remove Syntaxes/PostCSS.tmLanguage.

Thank you for doing this!

@maxinspace
Copy link
Contributor Author

@maxinspace maxinspace commented Oct 26, 2016

@hudochenkov: updated Readme, feel free to check and merge 👍

Thanks for sharing open-source offers on Cult Of Martians, haven't had a chance to do something for open-source community for a while :)

@ai
Copy link

@ai ai commented Oct 26, 2016

I asked my friends to test syntax file from Syntax-highlighting-for-SugarSS and they reported that it works very good.

@hudochenkov hudochenkov merged commit cec6d96 into hudochenkov:master Oct 26, 2016
@ai
Copy link

@ai ai commented Oct 26, 2016

@hudochenkov please ping me when you release it. I will write a tweets :)

@hudochenkov
Copy link
Owner

@hudochenkov hudochenkov commented Oct 26, 2016

Thank you, @maximlarionov!

@ai I've just released, but it's not updated in Package Control yet. It isn't available to users. I suppose it will be available in an hour.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants