search

atom / language-csharp

C# language support for Atom
Other
62 stars 53 forks source link

Remove redundant `source` scopes #37

Closed MaximSokolov closed 9 years ago

MaximSokolov commented 9 years ago

Fixes #35

50Wliu commented 9 years ago

@simurai

MaximSokolov commented 9 years ago

Requires changes after #36

MaximSokolov commented 9 years ago

Done

simurai commented 9 years ago

I think this change is good. AFAIK, source is only used to select a language, like .source.cs. But not for the rest, right?

MaximSokolov commented 9 years ago

I think this change is good. AFAIK, source is only used to select a language, like .source.cs. But not for the rest, right?

You are correct. I guess source is needed because of embedded code:

<script type="text/javascript">
    // source.js.embedded 
</script>
simurai commented 9 years ago

@MaximSokolov ok, thanks for cleaning it up.