Make final new line optional

Author: simonnilsson

README.md

@@ -171,6 +171,9 @@ export default {
   lineEnding: 'auto',
   // Control the line ending. See options at https://github.com/ryanve/eol
 
+  insertFinalNewline: true,
+  // Insert line ending at the end of output files
+
   locales: ['en', 'fr'],
   // An array of the locales in your applications
 

index.d.ts

@@ -105,6 +105,7 @@ export interface UserConfig {
     default?: (SupportedLexer | CustomLexer | LexerConfig)[];
   };
   lineEnding?: "auto" | "crlf" | "\r\n" | "cr" | "\r" | "lf" | "\n";
+  insertFinalNewline?: boolean;
   locales?: string[];
   namespaceSeparator?: string | false;
   output?: string;

src/transform.js

@@ -30,6 +30,7 @@ export default class i18nTransform extends Transform {
       keySeparator: '.',
       lexers: {},
       lineEnding: 'auto',
+      insertFinalNewline: true,
       locales: ['en', 'fr'],
       namespaceSeparator: ':',
       pluralSeparator: '_',
@@ -403,7 +404,9 @@ export default class i18nTransform extends Transform {
         ...this.options.yamlOptions,
       })
     } else {
-      text = JSON.stringify(contents, null, this.options.indentation) + '\n'
+      text =
+        JSON.stringify(contents, null, this.options.indentation) +
+        (this.options.insertFinalNewline ? '\n' : '')
       // Convert non-printable Unicode characters to unicode escape sequence
       // https://unicode.org/reports/tr18/#General_Category_Property
       text = text.replace(/[\p{Z}\p{Cc}\p{Cf}]/gu, (chr) => {

test/parser.test.js

@@ -1294,6 +1294,30 @@ describe('parser', () => {
       i18nextParser.end(fakeFile)
     })
 
+    it('supports insertFinalNewline', (done) => {
+      let result
+      const i18nextParser = new i18nTransform({
+        lineEnding: '\r\n',
+        insertFinalNewline: false,
+      })
+      const fakeFile = new Vinyl({
+        contents: Buffer.from("t('first')"),
+        path: 'file.js',
+      })
+
+      i18nextParser.on('data', (file) => {
+        if (file.relative.endsWith(enLibraryPath)) {
+          result = file.contents.toString()
+        }
+      })
+      i18nextParser.once('end', () => {
+        assert.equal(result, '{\r\n  "first": ""\r\n}')
+        done()
+      })
+
+      i18nextParser.end(fakeFile)
+    })
+
     it('parses Trans from js file with lexer override to JsxLexer', (done) => {
       let result
       const i18nextParser = new i18nTransform({