Skip to content

chore(templates): remover shamefully-hoist=false do .npmrc#26

Merged
rlueder merged 1 commit intomainfrom
chore/drop-shamefully-hoist-npmrc
Apr 24, 2026
Merged

chore(templates): remover shamefully-hoist=false do .npmrc#26
rlueder merged 1 commit intomainfrom
chore/drop-shamefully-hoist-npmrc

Conversation

@rlueder
Copy link
Copy Markdown
Member

@rlueder rlueder commented Apr 24, 2026

Resumo

`shamefully-hoist=false` é o default do pnpm e não é reconhecida pelo npm. Resultado: todo comando npm que lê `.npmrc` (`npm view`, `npm deprecate`, `npm publish` local, etc.) imprime:

```
npm warn Unknown project config "shamefully-hoist". This will stop working in the next major version of npm.
```

Remove a linha do template e da raiz do tooling. Fica só um header explicando a intenção; arquivo fica vazio-equivalente pra npm.

Cada consumer vai absorver a mudança via `precisa sync` ou manualmente.

Plano de teste

  • `npm view @precisa-saude/cli` não emite mais o warning na raiz do tooling.

A key `shamefully-hoist=false` e o default do pnpm e nao e reconhecida
pelo npm, que emite warning em todo comando que le .npmrc (`npm view`,
`npm deprecate`, etc.). Fica o header explicando a intencao, template
virazio-equivalente.

Cada consumer vai pegar a nova versao via `precisa sync` ou manualmente.
@github-actions
Copy link
Copy Markdown

Automated Review — Round 1

Summary

The changes made to the .npmrc and templates/dotfiles/npmrc files are appropriate, as they remove unnecessary settings that could lead to warnings from npm. No security issues or bugs are introduced.

Changes

  • Removed 'shamefully-hoist=false' to avoid unknown keys warning in npm.

✅ No issues found

Reviewed by OpenAI gpt-4o-mini (fallback) | 484 in / 71 out | $0.0001 — Round 1 of 2

@rlueder rlueder merged commit d489412 into main Apr 24, 2026
10 checks passed
@rlueder rlueder deleted the chore/drop-shamefully-hoist-npmrc branch April 24, 2026 21:54
rlueder added a commit to Precisa-Saude/datasus-dbc that referenced this pull request Apr 24, 2026
A key e o default do pnpm e nao e reconhecida pelo npm, que emite
warning em todo comando que le .npmrc. Alinha com o template do
tooling (ver Precisa-Saude/tooling#26).
rlueder added a commit to Precisa-Saude/datasus-sdk that referenced this pull request Apr 24, 2026
A key e o default do pnpm e nao e reconhecida pelo npm, que emite
warning em todo comando que le .npmrc. Alinha com o template do
tooling (ver Precisa-Saude/tooling#26).
rlueder added a commit to Precisa-Saude/datasus-parquet that referenced this pull request Apr 24, 2026
A key e o default do pnpm e nao e reconhecida pelo npm, que emite
warning em todo comando que le .npmrc. Alinha com o template do
tooling (ver Precisa-Saude/tooling#26).
rlueder added a commit to Precisa-Saude/datasus-viz that referenced this pull request Apr 24, 2026
A key e o default do pnpm e nao e reconhecida pelo npm, que emite
warning em todo comando que le .npmrc. Alinha com o template do
tooling (ver Precisa-Saude/tooling#26).
precisa-saude-release-bot Bot pushed a commit that referenced this pull request Apr 24, 2026
## [1.7.0](v1.6.1...v1.7.0) (2026-04-24)

### Features

* **cli:** tokens e blocos condicionais pro ci.yml render por repo ([#28](#28)) ([5e81d75](5e81d75)), closes [#if](https://github.com/Precisa-Saude/tooling/issues/if)

### Chores

* **templates:** remover deployments: write nao utilizado do ci.yml ([#27](#27)) ([68aed4b](68aed4b))
* **templates:** remover shamefully-hoist=false do .npmrc ([#26](#26)) ([d489412](d489412))
precisa-saude-release-bot Bot pushed a commit to Precisa-Saude/datasus-viz that referenced this pull request Apr 24, 2026
## [1.3.2](v1.3.1...v1.3.2) (2026-04-24)

### Bug Fixes

* **ci:** usar contexto do repo atual no pr-review-responder ([#17](#17)) ([fae51eb](fae51eb))

### Refactoring

* **site:** mover packages/site → site/ pra alinhar com o ecossistema ([#22](#22)) ([b11b96b](b11b96b))

### Documentation

* **site:** remover referencias obsoletas a datasus-brasil e branch PRE-206 ([#16](#16)) ([a2e73a5](a2e73a5))

### Chores

* **config:** precisa sync — governance/issue/PR templates ([#20](#20)) ([47ada22](47ada22))
* **config:** precisa sync — publishPackages + site wiring + template refresh ([#19](#19)) ([669f0f7](669f0f7)), closes [Precisa-Saude/tooling#28](Precisa-Saude/tooling#28)
* **config:** remover shamefully-hoist=false do .npmrc ([#18](#18)) ([2022034](2022034)), closes [Precisa-Saude/tooling#26](Precisa-Saude/tooling#26)
* **config:** siteSourcePath=packages/site/ + sync do _deploy-site ([#21](#21)) ([9046862](9046862)), closes [Precisa-Saude/tooling#29](Precisa-Saude/tooling#29)
Sign up for free to 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.

1 participant