Skip to content

Commit

Permalink
Merge pull request #1028 from ckeditor/remove-deprecations
Browse files Browse the repository at this point in the history
Other (release-tools): The `verifyPackagesPublishedCorrectly()` task is no longer available as its responsibility has been merged into the `publishPackages()` task.

MAJOR BREAKING CHANGE (release-tools): The `verifyPackagesPublishedCorrectly()` function is no longer available. Consider using the `publishPackages()` function, which includes its responsibility.
  • Loading branch information
pomek authored Oct 21, 2024
2 parents 372d283 + 3c6d10c commit 2afa211
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 161 deletions.
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.

0 comments on commit 2afa211

Please sign in to comment.