Conversation
… repo URLs Agent-Logs-Url: https://github.com/refined-github/octicons-plain-react/sessions/d00b1949-987c-4f58-8c9e-01e42ed5256e Co-authored-by: fregante <1402241+fregante@users.noreply.github.com>
fregante
changed the title
|
@SunsetTechuila new repo in the org! |
| with: | ||
| registry-url: 'https://registry.npmjs.org' | ||
| - uses: actions/checkout@v6 | ||
| - uses: actions/setup-node@v6 |
There was a problem hiding this comment.
Mottie/github-reserved-names#31 (comment)
Still relevant
|
@copilot use node 22 in setup-node |
Agent-Logs-Url: https://github.com/refined-github/octicons-plain-react/sessions/90141f67-eb82-4040-8628-d0eac2a3da20 Co-authored-by: fregante <1402241+fregante@users.noreply.github.com>
| - uses: actions/setup-node@v6 | ||
| with: | ||
| registry-url: 'https://registry.npmjs.org' | ||
| node-version: '22' |
There was a problem hiding this comment.
I would use
| node-version: '22' | |
| node-version: lts/* |
There was a problem hiding this comment.
This is a temporary workaround, we should prefer setting the engine field. Here we can't set it because it would be a breaking change and we're just following the source package's versioning.
They're not too far behind though, so soon: https://github.com/primer/octicons/blob/711ef28bd38aecc89380c5183caa91014f85c8f8/.github/workflows/ci.yml#L15
There was a problem hiding this comment.
Actually ☝️🤓 we should prefer devEngines.runtime. I never suggested it because fnm still doesn't support it 😭: Schniz/fnm#1433
id-token: writepermission for npm trusted publisher--provenanceflag tonpm publishactions/setup-node