Build script fails on Windows due to CRLF line endings in pnpm-workspace.yaml
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 85/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- node.js, typescript, yaml
- Bereich
- build-system, tooling
Rechercherichtung
Beginne mit scripts/verify-packages-versions.js, insbesondere mit dem YAML-Parsing in der Nähe der verlinkten Zeile. Prüfe, wie das Skript pnpm-workspace.yaml mit CRLF-Zeilenenden verarbeitet, und führe anschließend den Build mit diesem Zeilenendenformat aus. Fertig ist die Aufgabe, wenn die Paketlisten des Workspaces korrekt verglichen werden und der Build unter Windows erfolgreich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/browser
SDK Version
N/A (monorepo root)
Framework Version
Node.js 24 on Windows
Link to Sentry event
No response
Reproduction Example/SDK Setup
No response
Steps to Reproduce
- Clone the sentry-javascript repository on a Windows machine.
- Ensure Git checks out pnpm-workspace.yaml with CRLF (\r\n) line endings.
- Run pnpm install and then pnpm run build
Expected Result
The build processes the workspace packages and runs successfully.
Actual Result
The build crashes immediately with the following error:
[INCONSISTENT WORKSPACES]: The packageslist in pnpm-workspace.yaml and theworkspaces list in package.json must contain the same packages in the same order.
I guess this happens because scripts/verify-packages-versions.js rigidly splits the YAML file using .split('\n'). On Windows, this reads the target string as 'packages:\r', causing the match to fail, returning an empty array and triggering the error.
Additional Context
Suggested Solution:
Update scripts/verify-packages-versions.js to handle both line endings by using a regex split: .split(/\r?\n/)
I'm happy to open a PR though.
Priority
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 8.8k
- Forks
- 1.9k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 664
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus getsentry/sentry-javascript
-
Bug Package: node Spans vercel ai
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
getsentry/sentry-javascript#26079 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Bug Node.js
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
getsentry/sentry-javascript#25950 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
normalize() throws when a getter throws a Symbol or a null-prototype objectEvtl. vergeben @rjmcgee53192 hat das vor 4 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
getsentry/sentry-javascript#25920 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Only create Nuxt database and storage spans when there is a parent spanEvtl. vergeben @nicohrubec hat das vor 5 Tagen übernommen. OffenImprovement Nuxt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
getsentry/sentry-javascript#25805 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Only create Cloudflare storage and D1 db spans when there is a parent spanEvtl. vergeben @nicohrubec hat das vor 5 Tagen übernommen. OffenCloudflare Workers Improvement Spans
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
getsentry/sentry-javascript#25804 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in getsentry/sentry-javascript
Ähnliche Issues
-
test: claim-conflict drop leaves both facts active and is never re-proposed (untested review cell)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
Maintainer antworten meist innerhalb von 1 Tag
-
[Feature]: Usage page should show API-equivalent cost and say when a connected plan shows info onlyOffenenhancement from-x
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
nightly-labs/openbot#4867 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
egonSchiele/agency-lang#1377 ·
Maintainer antworten meist innerhalb von 1 Tag
-
normalizePath removes the Windows device namespace prefixEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
caprover/caprover-frontend#273 ·
Maintainer antworten meist innerhalb von 1 Tag