Ticket #3890 (closed enhancement: fixed)

Opened 7 years ago

Last modified 7 years ago

Add "mjs" to the JavaScript syntax line

Reported by: dandv Owned by: zaytsev
Priority: minor Milestone: 4.8.21
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

Starting with Node 8.5.0, there is a new extension for ​native JavaScript modules: .mjs

It would be great to add "mjs" to the appropriate line in Syntax:

file ..\*\\.(js|mjs|JS)$ JavaScript\sProgram
include js.syntax

(I'd happily submit a patch if the project were hosted somewhere that made it easy to do so, such as on GitHub?)

Change History

comment:1 Changed 7 years ago by zaytsev

  • Priority changed from major to minor
  • Owner set to zaytsev
  • Status changed from new to accepted
  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.21

Branch: 3890_syntax_mjs.
Initial changeset:a778825d0efe96d486a01abbbf2a3d31b2dae65c

comment:2 Changed 7 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Branch state changed from on review to approved

comment:3 Changed 7 years ago by zaytsev

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

Merged to master: 38ce185cd3f09e0d156c3e2202b628c1c8deaed8

comment:4 Changed 7 years ago by zaytsev

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.