microsoft / DevSkim

DevSkim is a set of IDE plugins, language analyzers, and rules that provide security "linting" capabilities.
MIT License
902 stars 115 forks source link

Atom support #3

Open matthewberryman opened 7 years ago

matthewberryman commented 7 years ago

It would be nice if you could support Atom as well.

joshbw commented 7 years ago

Atom is definitely on our radar, and we should be able to re-use a lot of the VS Code logic. I'll have to dig into its extension model, but I'll start investigating

matthewberryman commented 7 years ago

Great, thanks @joshbw. I'd devote some time to this myself but I'm stretched across a lot of paid projects at the moment.

joshbw commented 7 years ago

@matthewberryman - after spending a little freetime with Atom's expansion model, it looks non-trivial to get the TypeScript of the VSCode plugin to play nice in the Coffescript environment of Atom (it looks like atom is directly loading .coffee files, instead of the transpiled javascript which would have made things pretty easy typescript wise). There might be a simple trick to it, but so far I haven't spotted it. I got a bunch of travel coming up that is going to cut into my productivity, and we have some exciting enhancements in the hopper for our existing clients that I need to get hammered out so I might not get a chance to really tackle an Atom client until early summer. If you know of a decent typescript -> coffeescript converter that may make things a bit easier, but so far I've only found converters that go the other direction (the general momentum seems to be more behind TS than CS in that regard)

steve-hawkins commented 7 years ago

TS > JS > CS... not the nicest, but might be a quick win?

passcod commented 6 years ago

Has any progress been made on this? And, could you not use coffeescript js embedding support as a cheap solution? E.g.

``​`
/* put transpiled JS here */
``​`

It will compile to:

/* put transpile JS here */
;

Should be pretty safe.