search

jfmengels / node-elm-review

CLI for elm-review
https://package.elm-lang.org/packages/jfmengels/elm-review/latest/
BSD 3-Clause "New" or "Revised" License
47 stars 25 forks source link

Sync Support #167

Closed lishaduck closed 1 week ago

lishaduck commented 1 week ago

Sync support with jfmengels/elm-review#174 Also, fix a link in the readme.

As a random human, do you mind deleting node-elm-review:master? It's confusing. As is using elm-review:master rather than main.

jfmengels commented 1 week ago

Sorry for the confusion between master and main! :disappointed: I've removed it now. I'll pull the fix from this PR without the link change from main to master. Let me know if I was mistaken to do so.

lishaduck commented 1 week ago

Sorry for the confusion between master and main! 😞 I've removed it now. I'll pull the fix from this PR without the link change from main to master. Let me know if I was mistaken to do so.

This link fix is still valid. elm-review uses master, not main. It would probably be less confusing to also rename it there as well.

jfmengels commented 1 week ago

Oh right, that was indeed confusing. Ok, well time for me to change it over there!