Skip to content

Commit

Permalink
Merge pull request #1029 from ckeditor/ci/epic/3789
Browse files Browse the repository at this point in the history
Feature (translations): Introduced `moveTranslations()` function to move requested translations between packages. It removes contexts and translated messages from language files (`*.po` files) from the source package and adds (or overwrites) them in the destination package.

Feature (translations): Introduced `synchronizeTranslations()` function to synchronize translations (stored in `*.po` files in packages) with context files. It validates translation contexts against the source messages and performs synchronization by removing unused entries, adding missing entries, and creating missing `*.po` files. If the `options.validateOnly` flag is set, no translation files are updated, and only validation is performed.

Other (release-tools): The `verifyPackagesPublishedCorrectly()` task is no longer available as its responsibility has been merged into the `publishPackages()` task.

MAJOR BREAKING CHANGE (transifex): Removed the `@ckeditor/ckeditor5-dev-transifex` package as it is no longer used.

MAJOR BREAKING CHANGE (translations): The `cleanPoFileContent()` and `createDictionaryFromPoFileContent()` functions are no longer available due to removal integration with the Transifex service.

MAJOR BREAKING CHANGE (release-tools): The `verifyPackagesPublishedCorrectly()` function is no longer available. Consider using the `publishPackages()` function, which includes its responsibility.
  • Loading branch information
psmyrek authored Oct 22, 2024
2 parents 44d9ff5 + a954a55 commit a53514d
Show file tree
Hide file tree
Showing 59 changed files with 2,925 additions and 5,398 deletions.
5 changes: 2 additions & 3 deletions .eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@ module.exports = {

// ESLint does not understand `import ... with { ... }`.
// See: https://github.com/eslint/eslint/discussions/15305.
'packages/ckeditor5-dev-ci/lib/data/index.js',
'packages/ckeditor5-dev-transifex/lib/data/index.js'
'packages/ckeditor5-dev-ci/lib/data/index.js'
],
rules: {
'no-console': 'off',
Expand All @@ -43,7 +42,7 @@ module.exports = {
'./packages/typedoc-plugins/**/*'
],
rules: {
'ckeditor5-rules/require-file-extensions-in-imports': 'off',
'ckeditor5-rules/require-file-extensions-in-imports': 'off'
}
}
]
Expand Down
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ This repository is a monorepo. It contains multiple npm packages.
| [`@ckeditor/ckeditor5-dev-docs`](/packages/ckeditor5-dev-docs) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-docs.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-docs) |
| [`@ckeditor/ckeditor5-dev-release-tools`](/packages/ckeditor5-dev-release-tools) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-release-tools.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-release-tools) |
| [`@ckeditor/ckeditor5-dev-tests`](/packages/ckeditor5-dev-tests) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-tests.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-tests) |
| [`@ckeditor/ckeditor5-dev-transifex`](/packages/ckeditor5-dev-transifex) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-transifex.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-transifex) |
| [`@ckeditor/ckeditor5-dev-utils`](/packages/ckeditor5-dev-utils) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-utils.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-utils) |
| [`@ckeditor/ckeditor5-dev-translations`](/packages/ckeditor5-dev-translations) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-translations.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-translations) |
| [`@ckeditor/ckeditor5-dev-web-crawler`](/packages/ckeditor5-dev-web-crawler) | [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-web-crawler.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-web-crawler) |
Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
"@ckeditor/ckeditor5-dev-bump-year": "^44.2.1",
"@inquirer/prompts": "^6.0.0",
"@listr2/prompt-adapter-inquirer": "^2.0.16",
"@octokit/rest": "^21.0.0",
"eslint": "^8.21.0",
"eslint-config-ckeditor5": "^8.0.0",
"fs-extra": "^11.0.0",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
msgid ""
msgstr ""
"Language-Team: German (https://app.transifex.com/ckeditor/teams/11143/de/)\n"
"Language: de\n"
"Plural-Forms: nplurals=2; plural=(n != 1);\n"
"Content-Type: text/plain; charset=UTF-8\n"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
msgid ""
msgstr ""
"Language-Team: Polish (https://app.transifex.com/ckeditor/teams/11143/pl/)\n"
"Language: pl\n"
"Plural-Forms: nplurals=4; plural=(n==1 ? 0 : (n%10>=2 && n%10<=4) && (n%100<12 || n%100>14) ? 1 : n!=1 && (n%10>=0 && n%10<=1) || (n%10>=5 && n%10<=9) || (n%100>=12 && n%100<=14) ? 2 : 3);\n"
"Content-Type: text/plain; charset=UTF-8\n"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
msgid ""
msgstr ""
"Language-Team: Polish (https://app.transifex.com/ckeditor/teams/11143/pl/)\n"
"Language: pl\n"
"Plural-Forms: nplurals=4; plural=(n==1 ? 0 : (n%10>=2 && n%10<=4) && (n%100<12 || n%100>14) ? 1 : n!=1 && (n%10>=0 && n%10<=1) || (n%10>=5 && n%10<=9) || (n%100>=12 && n%100<=14) ? 2 : 3);\n"
"Content-Type: text/plain; charset=UTF-8\n"
Expand Down
1 change: 0 additions & 1 deletion packages/ckeditor5-dev-release-tools/lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ export { default as saveChangelog } from './utils/savechangelog.js';
export { default as executeInParallel } from './utils/executeinparallel.js';
export { default as validateRepositoryToRelease } from './utils/validaterepositorytorelease.js';
export { default as checkVersionAvailability } from './utils/checkversionavailability.js';
export { default as verifyPackagesPublishedCorrectly } from './tasks/verifypackagespublishedcorrectly.js';
export { default as getNpmTagFromVersion } from './utils/getnpmtagfromversion.js';
export { default as isVersionPublishableForTag } from './utils/isversionpublishablefortag.js';
export { default as provideToken } from './utils/providetoken.js';
Expand Down

This file was deleted.

9 changes: 0 additions & 9 deletions packages/ckeditor5-dev-release-tools/tests/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ import {
import executeInParallel from '../lib/utils/executeinparallel.js';
import validateRepositoryToRelease from '../lib/utils/validaterepositorytorelease.js';
import checkVersionAvailability from '../lib/utils/checkversionavailability.js';
import verifyPackagesPublishedCorrectly from '../lib/tasks/verifypackagespublishedcorrectly.js';
import getNpmTagFromVersion from '../lib/utils/getnpmtagfromversion.js';
import isVersionPublishableForTag from '../lib/utils/isversionpublishablefortag.js';
import provideToken from '../lib/utils/providetoken.js';
Expand All @@ -50,7 +49,6 @@ vi.mock( '../lib/tasks/push' );
vi.mock( '../lib/tasks/publishpackages' );
vi.mock( '../lib/tasks/updateversions' );
vi.mock( '../lib/tasks/cleanuppackages' );
vi.mock( '../lib/tasks/verifypackagespublishedcorrectly' );
vi.mock( '../lib/utils/versions' );
vi.mock( '../lib/utils/getnpmtagfromversion' );
vi.mock( '../lib/utils/changelog' );
Expand Down Expand Up @@ -243,13 +241,6 @@ describe( 'dev-release-tools/index', () => {
} );
} );

describe( 'verifyPackagesPublishedCorrectly()', () => {
it( 'should be a function', () => {
expect( verifyPackagesPublishedCorrectly ).to.be.a( 'function' );
expect( index.verifyPackagesPublishedCorrectly ).to.equal( verifyPackagesPublishedCorrectly );
} );
} );

describe( 'isVersionPublishableForTag()', () => {
it( 'should be a function', () => {
expect( isVersionPublishableForTag ).to.be.a( 'function' );
Expand Down

This file was deleted.

6 changes: 0 additions & 6 deletions packages/ckeditor5-dev-transifex/CHANGELOG.md

This file was deleted.

16 changes: 0 additions & 16 deletions packages/ckeditor5-dev-transifex/LICENSE.md

This file was deleted.

17 changes: 0 additions & 17 deletions packages/ckeditor5-dev-transifex/README.md

This file was deleted.

Loading

0 comments on commit a53514d

Please sign in to comment.