phjardas / taktische-zeichen

Generator für taktische Zeichen nach der DV 102

https://taktische-zeichen.dev

MIT License

15 stars 4 forks source link

Fix typo in cli example #53

Closed white-gecko closed 9 months ago

white-gecko commented 9 months ago

❤️ Vielen Dank für deinen PR!

Ein typo im cli example:

--einheit grupp

--einheit gruppe

✅ Checkliste

Bitte stelle sicher, dass du die folgenden Aufgaben für deinen PR erledeigt hast:

[x] Changeset erstellt (siehe Contributors' Guide).

Verwende bitte die folgenden Versionsanpassungen:
- Fehler korrigiert ➡️ patch
- Neues Symbol hinzugefügt ➡️ minor
- Nicht rückwärtskompatible Änderung ➡️ major

Ich hoffe, ich habe das richtig gemacht.

[ ] Dokumentation angepasst mit npm run update-docs.

Lief leider nicht.

⚖️ Lizenz

Durch das Einreichen dieses PRs erklärst du dich damit einverstanden, dass dein Code im Rahmen dieser Bibliothek unter einer MIT-Lizenz kostenlos veröffentlicht wird.

changeset-bot[bot] commented 9 months ago

🦋 Changeset detected

Latest commit: 996c68571d01da67693fa6a295f1169c3690021f

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

netlify[bot] commented 9 months ago

Deploy Preview for taktische-zeichen ready!

Name	Link
Latest commit	996c68571d01da67693fa6a295f1169c3690021f
Latest deploy log	https://app.netlify.com/sites/taktische-zeichen/deploys/65ad24dfd3bfbf00081fa7ba
Deploy Preview	https://deploy-preview-53--taktische-zeichen.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

phjardas commented 9 months ago

Hey @white-gecko, vielen Dank für deinen PR! Kannst du mir sagen, was bei npm run update-docs nicht funktioniert?

white-gecko commented 9 months ago

Dann habe ich gestern wohl einen entscheidenden Schritt ausgelassen. Jetzt habe nochmal entsprechend der CONTRIBUTING.md in einem GitHub Code Space folgendes ausgeführt mit folgenden Fehlern:

$ npm ci # lief durch
$ npm bootstrap
Unknown command: "bootstrap"

Did you mean this?
    npm run bootstrap # run the "bootstrap" package script

To see a list of supported npm commands, run:
  npm help
$ npm run bootstrap

> taktische-zeichen-parent@0.0.0 bootstrap
> lerna bootstrap

lerna notice cli v4.0.0
lerna info versioning independent
lerna info Bootstrapping 5 packages
lerna info Installing external dependencies
lerna ERR! npm install exited 7 in 'taktische-zeichen-web-component'
lerna ERR! npm install stderr:
npm WARN deprecated querystring@0.2.0: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm ERR! code 7
npm ERR! path /workspaces/taktische-zeichen-dev/packages/web-component/node_modules/lmdb
npm ERR! command failed
npm ERR! command sh -c node-gyp-build
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using node-gyp@5.1.1
npm ERR! gyp info using node@20.11.0 | linux | x64
npm ERR! gyp info find Python using Python version 3.10.13 found at "/home/codespace/.python/current/bin/python"
npm ERR! gyp http GET https://nodejs.org/download/release/v20.11.0/node-v20.11.0-headers.tar.gz
npm ERR! gyp http 200 https://nodejs.org/download/release/v20.11.0/node-v20.11.0-headers.tar.gz
npm ERR! gyp http GET https://nodejs.org/download/release/v20.11.0/SHASUMS256.txt
npm ERR! gyp http 200 https://nodejs.org/download/release/v20.11.0/SHASUMS256.txt
npm ERR! gyp ERR! UNCAUGHT EXCEPTION 
npm ERR! gyp ERR! stack TypeError: Cannot assign to read only property 'cflags' of object '#<Object>'
npm ERR! gyp ERR! stack     at createConfigFile (/workspaces/taktische-zeichen-dev/node_modules/node-gyp/lib/configure.js:118:21)
npm ERR! gyp ERR! stack     at /workspaces/taktische-zeichen-dev/node_modules/node-gyp/lib/configure.js:85:9
npm ERR! gyp ERR! stack     at /workspaces/taktische-zeichen-dev/node_modules/node-gyp/node_modules/mkdirp/index.js:30:20
npm ERR! gyp ERR! stack     at FSReqCallback.oncomplete (node:fs:191:23)
npm ERR! gyp ERR! System Linux 6.2.0-1018-azure
npm ERR! gyp ERR! command "/usr/local/share/nvm/versions/node/v20.11.0/bin/node" "/workspaces/taktische-zeichen-dev/node_modules/.bin/node-gyp" "rebuild"
npm ERR! gyp ERR! cwd /workspaces/taktische-zeichen-dev/packages/web-component/node_modules/lmdb
npm ERR! gyp ERR! node -v v20.11.0
npm ERR! gyp ERR! node-gyp -v v5.1.1
npm ERR! gyp ERR! Node-gyp failed to build your package.
npm ERR! gyp ERR! Try to update npm and/or node-gyp and if it does not help file an issue with the package author.

npm ERR! A complete log of this run can be found in: /home/codespace/.npm/_logs/2024-01-21T11_53_21_836Z-debug-0.log
lerna ERR! npm install exited 7 in 'taktische-zeichen-web-component'
$ npm run build

> taktische-zeichen-parent@0.0.0 build
> lerna run build

lerna notice cli v4.0.0
lerna info versioning independent
lerna info Executing command in 4 packages: "npm run build"
lerna info run Ran npm script 'build' in 'taktische-zeichen-core' in 6.7s:

> taktische-zeichen-core@0.7.0 build
> npm run build:esm && npm run build:cjs

> taktische-zeichen-core@0.7.0 build:esm
> tsc

> taktische-zeichen-core@0.7.0 build:cjs
> tsc -p tsconfig-cjs.json

lerna ERR! npm run build exited 127 in 'taktische-zeichen-web-component'
lerna ERR! npm run build stdout:

> taktische-zeichen-web-component@0.7.0 build
> npm run build:esm && npm run build:cjs

> taktische-zeichen-web-component@0.7.0 build:esm
> tsc

lerna ERR! npm run build stderr:
sh: 1: tsc: not found
lerna ERR! npm run build exited 127 in 'taktische-zeichen-web-component'
lerna WARN complete Waiting for 1 child process to exit. CTRL-C to exit immediately.
$ npx changeset # lief durch
$ npm run update-docs # lief jetzt auch durch

phjardas commented 9 months ago

Danke für dein Feedback! Ich schätze der Fehler kommt daher dass das Projekt noch mit einer alten Version von Node.js (16 🤯) baut, dein CoseSpace aber mit Version 20 läuft. Ich habe noch nie CodeSpace benutzt... Kannst du dort irgendwie die Version von Node.js einstellen?

Durch update-docs sollte noch eine Änderung an der readme entstehen, kannst du die bitte auch in den PR pushen?

Dankeschön! 🤗

phjardas commented 9 months ago

Ich habe die Version von Node.js in main auf 21 aktualisiert. Bitte rebase deinen Branch und probiere es damit erneut. :-)

phjardas commented 9 months ago

@all-contributors please add @white-gecko for bugfix

allcontributors[bot] commented 9 months ago

@phjardas

I couldn't determine any contributions to add, did you specify any contributions? Please make sure to use valid contribution names.

phjardas commented 9 months ago

@all-contributors please add @white-gecko for code

allcontributors[bot] commented 9 months ago

@phjardas

File LICENSE.md was not found in the repository (phjardas/taktische-zeichen).

phjardas commented 9 months ago

@all-contributors please add @white-gecko for code

white-gecko commented 9 months ago

Sorry, ich hatte das Buildsystem nicht ganz verstanden und auch nicht, dass die README generiert wird. Sollte jetzt alles passen, habe ge-rebased und gesqashed.

phjardas commented 9 months ago

@all-contributors please add @white-gecko for code

allcontributors[bot] commented 9 months ago

@phjardas

@white-gecko already contributed before to code

phjardas commented 9 months ago

Vielen lieben Dank für deinen Beitrag!