You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, this plugin uses os.EOL as the new line character when generating new files (such as in reference.ts). This can cause problems in cross platform development and in source control management as the generated reference files will be different depending on whether they were generated on Windows or Linux/Mac. Grunt has the ability to optionally specify the line feed character (still defaults to os.EOL) with grunt.util.linefeed (gruntjs/grunt-contrib-concat#20). To take advantage of this, all you would need to do is update the references to os.EOL. I can do this work and create a pull request, but I want to make sure you are OK with this before spending the time (there are 14 references throughout the entire code base)
The text was updated successfully, but these errors were encountered:
Should I update the files in tasks-internal? Reading comments from another issue, it looks like tasks-internal is used as the internal compiler you use to turn the grunt-ts typescript into javascript
no need. Your understanding is correct. It will automatically get updated the next time we run grunt update to update the LKG. Looking forward to the PR!
Currently, this plugin uses os.EOL as the new line character when generating new files (such as in reference.ts). This can cause problems in cross platform development and in source control management as the generated reference files will be different depending on whether they were generated on Windows or Linux/Mac. Grunt has the ability to optionally specify the line feed character (still defaults to os.EOL) with grunt.util.linefeed (gruntjs/grunt-contrib-concat#20). To take advantage of this, all you would need to do is update the references to os.EOL. I can do this work and create a pull request, but I want to make sure you are OK with this before spending the time (there are 14 references throughout the entire code base)
The text was updated successfully, but these errors were encountered: