search

marigold-ui / marigold

Design System based on react-aria and Tailwind CSS
https://marigold-ui.io
MIT License
101 stars 7 forks source link

chore: upgrade yarn to v2 (berry) #868

Closed sebald closed 3 years ago

sebald commented 3 years ago
changeset-bot[bot] commented 3 years ago

⚠️ No Changeset found

Latest commit: 581f61abf3ac4d11d8e4b714dad7dfd31a53866a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 581f61abf3ac4d11d8e4b714dad7dfd31a53866a on yarn-v2 into b9a770772b5666636d4604c3cac1afb454572383 on main.

sebald commented 3 years ago

Yarn overall ausführen! Nice vergessen :D

sebald commented 3 years ago

yarn lint doesnt work for me

Need an error message to reproduce. Did you do a clean install of the node modules?

ti10le commented 3 years ago

@sebald bekomme keine Meldung bei yarn lint: image Sonst funktioniert so wie ich das sehen kann alles.

ti10le commented 3 years ago

@viktoria-schwarz do you have any problems with this changes?

sebald commented 3 years ago

@ti10le Versuch mal mit --verbose flag 🤔

viktoria-schwarz commented 3 years ago

Bei mir funktioniert alles außer dass @marigold/theme-b2b in den docs/src/pages nicht gefunden wird, aber das wird wohl an was anderem liegen, der Rest geht :+1:

viktoria-schwarz commented 3 years ago

@ti10le das ist doch normal dass wenn es keine Fehler gibt bei yarn lint kein output kommt?

viktoria-schwarz commented 3 years ago

sorry für den merge quatsch, kriege das grad nicht richtig hin :( aber wegen mir können wir das mergen, den Error dass das Theme nicht gefunden wird werd ich noch irgendwie fixen können wenn es ja bei euch funktioniert. Der Rest passt ja bei mir auch

sebald commented 3 years ago

Bei mir funktioniert alles außer dass @marigold/theme-b2b in den docs/src/pages nicht gefunden wird, aber das wird wohl an was anderem liegen, der Rest geht :+1:

@viktoria-schwarz ja, wie gesagt, das liegt daran, dass die packages gerade gebaut sein mussen. Wenn man das öffnen den build machen will (was wir sollten) muss man webpack noch erklären wo das Zeug liegt (ähnlich wie bei storybook)

ti10le commented 3 years ago

Also bei mir kam immer ne Meldung: Erfolgreich doneohne Fehler. Jetzt kommt halt gar nichts. Vllt funktioniert es trzd oder was anderes stimmt nicht wenn es bei euch beiden geht 🤷🏽‍♂️

sebald commented 3 years ago

Lass es doch mal im Main Branch laufen, dann weißt du es. Also das Konto command läuft schon noch. Sieht man daran wenn man extra lint Fehler im Code lässt.