Exit code when translations:collect reports an error

[pomek]

Suggested merge commit message (convention)

Other (env): Set the proper exit code when found an error when collecting translations. Closes ckeditor/ckeditor5#10123.

[coveralls]

Coverage increased (+0.02%) to 87.17% when pulling 03983fb on ck/10123 into 3f5fe79 on master.

[mlewand]

The solution looks to be working just fine. I just have one question.

[mlewand]

This part is repeated in all three functions. Wouldn't it be better to do this logic outside of assert* functions once?

Instead of

const foundErrors = [
		assertNoMissingContext( { packageContexts, sourceMessages, logger } ),
		assertAllContextUsed( { packageContexts, sourceMessages, logger } ),
		assertNoRepeatedContext( { packageContexts, logger } )
	].filter( errors => errors.length ).length > 0;

one could do:

const errors = [].concat(
		assertNoMissingContext( { packageContexts, sourceMessages, logger } ),
		assertAllContextUsed( { packageContexts, sourceMessages, logger } ),
		assertNoRepeatedContext( { packageContexts, logger } )
	);

// (...)

for ( const error of errors ) {
	logger.error( error );

	process.exitCode = 1;
}

[pomek]

Done in 03983fb.

[mlewand]

Looking good 👌