Skip to content

chore: remove duplicate tscheck from the publish script#16702

Merged
nicolo-ribaudo merged 1 commit intobabel:mainfrom
JLHwung:remove-duplicate-tscheck-prerequisite
Jul 31, 2024
Merged

chore: remove duplicate tscheck from the publish script#16702
nicolo-ribaudo merged 1 commit intobabel:mainfrom
JLHwung:remove-duplicate-tscheck-prerequisite

Conversation

@JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Jul 30, 2024

Q                       A
Fixed Issues? Remove duplicate tscheck from the publish script
License MIT

$(MAKEJS) prepublish-prepare-dts already runs $(MAKEJS) tscheck:

babel/Makefile.source.mjs

Lines 295 to 299 in 44efb5f

target["prepublish-prepare-dts"] = function () {
target["clean-ts"]();
target["tscheck"]();
target["prepublish-prepare-dts-no-clean"]();
};

@JLHwung JLHwung changed the title chore: $(MAKEJS) prepublish-prepare-dts already runs makejs tscheck chore: remove duplicate tscheck from the publish script Jul 30, 2024
@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/57552

@nicolo-ribaudo nicolo-ribaudo merged commit 66f1d47 into babel:main Jul 31, 2024
@nicolo-ribaudo nicolo-ribaudo deleted the remove-duplicate-tscheck-prerequisite branch July 31, 2024 08:58
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Nov 2, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants