Skip to content

chore!: switch to @node-core scope - #700

Merged
aduh95 merged 1 commit into
mainfrom
scoped-package-name
Sep 24, 2023
Merged

chore!: switch to @node-core scope#700
aduh95 merged 1 commit into
mainfrom
scoped-package-name

Conversation

@aduh95

Copy link
Copy Markdown
Contributor

Not sure this is ready yet, we may need a new NPM_TOKEN on the repo secret.

Refs: nodejs/TSC#1178

@codecov

codecovBot commented Jun 1, 2023

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (517a259) 83.03% compared to head (c5f695f) 83.03%.

Additional details and impacted files
@@ Coverage Diff @@## main #700 +/- ##
=======================================
Coverage 83.03% 83.03% =======================================
Files 37 37 Lines 4178 4178 =======================================
Hits 3469 3469 Misses 709 709 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MoLow

MoLow commented Jun 1, 2023

Copy link
Copy Markdown
Member

we should probably also publish one last version to https://www.npmjs.com/package/node-core-utils with a output/warning saying it moved

@aduh95

Copy link
Copy Markdown
ContributorAuthor

we should probably also publish one last version to https://www.npmjs.com/package/node-core-utils with a output/warning saying it moved

I think we can npm deprecate it without publishing a new version.

@targos

Copy link
Copy Markdown
Member

The current NPM_TOKEN should work. I can replace it with a granular one when the package exists on npm.

MoLow
MoLow previously approved these changes Jun 2, 2023
@MoLow
MoLow dismissed their stale reviewJune 2, 2023 12:16

Readme needs an update

@targos

Copy link
Copy Markdown
Member

release-please workflow needs an update as well.

@aduh95
aduh95 merged commit f334713 into mainSep 24, 2023
@github-actionsgithub-actionsBot mentioned this pull request Sep 24, 2023
@aduh95
aduh95 deleted the scoped-package-name branch September 24, 2023 09:21
nodejs-github-bot pushed a commit to nodejs/node that referenced this pull request Sep 25, 2023
Refs: https://github.com/nodejs/node-core-utils/releases/tag/v4.0.0
PR-URL: #49851
Refs: nodejs/node-core-utils#700
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
ruyadorno pushed a commit to nodejs/node that referenced this pull request Sep 28, 2023
Refs: https://github.com/nodejs/node-core-utils/releases/tag/v4.0.0
PR-URL: #49851
Refs: nodejs/node-core-utils#700
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
ruyadorno pushed a commit to nodejs/node that referenced this pull request Sep 28, 2023
Refs: https://github.com/nodejs/node-core-utils/releases/tag/v4.0.0
PR-URL: #49851
Refs: nodejs/node-core-utils#700
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@aduh95@MoLow@targos@VoltrexKeyva