search

atom / language-todo

TODO highlighting package for Atom
Other
80 stars 66 forks source link

Adding scala support. #25

Closed iammichiel closed 9 years ago

iammichiel commented 9 years ago

Adding scala syntax :

// TODO: $0
/* TODO: $0 */
kevinsawicki commented 9 years ago

Is there a reason to enable both // and /* style TODO comments in scala? Would just // not work?

iammichiel commented 9 years ago

As both comment syntaxes are supported in scala, why not activate it in both cases? // will probably be the most used.

kevinsawicki commented 9 years ago

As both comment syntaxes are supported in scala, why not activate it in both cases?

It seems a little confusing to have 2 todo snippets listed in autocomplete for scala files.

Are you okay with just going with the // case to start?

iammichiel commented 9 years ago

Yes, good for me. I'll update my commit.

kevinsawicki commented 9 years ago

Awesome, thanks for this, this will be included in the next Atom release, 0.202.0