Open ThomasWeiser opened 8 years ago
@deadfoxygrandpa ping
Would adding support for "port module" and "effects module" be too much effort?
Added a commit with:
port
and effect
module syntax.Elm.YAML-tmLanguage
and Elm.tmLanguage
Example of valid module declarations in 0.17:
effect module WebSocket where { command = MyCmd, subscription = MySub } exposing (
send
, listen
, keepAlive
)
port module Spelling exposing (..)
module WebSocket.LowLevel exposing ( WebSocket
, open, Settings
, send, close, closeWith
, bytesQueued
, BadOpen(..), BadClose(..), BadSend(..)
)
module Main
0.16:
module Config ( title ) where
(Please note, that at the time of this writing GitHub highlights the code blocks above according to 0.16 rules. This repo is the source of GitHub's Elm rules. So merging this PR should change also the highlighting here.)
In case anyone wants to quickly apply this manually, you can try the following - which worked for me on OS X. You'll need to adjust the paths as appropriate
cd /Users/glen/Library/Application\ Support/Sublime\ Text\ 3/Packages/Elm\ Language\ Support
curl https://github.com/deadfoxygrandpa/Elm.tmLanguage/pull/101.patch -L | patch -p1
For @glenjamin's approach on Ubuntu, the directory to run the curl command from (at least for my specific st3 install):
~/.config/sublime-text-3/Packages/Elm\ Language\ Support
@deadfoxygrandpa this is a huge community pain point, and Sublime is now the only editor not updated for 0.17's breaking syntax changes...are you still up for maintaining this repo?
@deadfoxygrandpa Please consider moving this project to elm-community organization. What do you think @rtfeldman, would having support for major editors under elm-community make sense?
This has already been moved to elm-community. Deadfoxygrandpa has been unresponsive for about a month now
On Monday, May 30, 2016, Esad Hajdarevic notifications@github.com wrote:
@deadfoxygrandpa https://github.com/deadfoxygrandpa Please consider moving this project to elm-community organization. What do you think @rtfeldman https://github.com/rtfeldman, would having support for major editors under elm-community make sense?
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/deadfoxygrandpa/Elm.tmLanguage/pull/101#issuecomment-222474749, or mute the thread https://github.com/notifications/unsubscribe/ABFh_hNyPay6QXnO5qawdyJbFVwSdsmhks5qGs1MgaJpZM4IUQr9 .
Cheers, Noah
noah@astraldynamics.co.uk Astral Dynamics Cyf Festri Cefn Y Waen, Deiniolen, Caernarfon, Gwynedd LL55 3NB Company number 07726862 Registered in England and Wales
@eeue56 Should we make PR to https://github.com/wbond/package_control_channel to have the default package switch to the new repo?
Just opened a PR to adopt the package by elm-community.
@ThomasWeiser probably makes sense to do the same for linguist, to update GH, yeah?
Opened PR at github/linguist.
Grammar now accepts both syntax variations:
Elm 0.16:
module Queue (..) where
Elm 0.17:module Queue exposing (..)