Mickhat / FloBot

7 stars 1 forks source link

Put it into the git ignore #79

Closed filip326 closed 1 month ago

filip326 commented 7 months ago
          simply because it's not in the gitignore

Originally posted by @Barsch2006 in https://github.com/Mickhat/FloBot/issues/78#issuecomment-1889843205

chfuchte commented 7 months ago

we should check, if its not easier with the package-lock 'cause npm ciin the action

chfuchte commented 7 months ago

@filip326 https://chat.openai.com/share/91b580c1-a748-4746-838a-590e3b1d3e43

chfuchte commented 7 months ago

I talked shit. It has a proper reason to exist on github

filip326 commented 7 months ago

We should check if its possible to use the yarn.lock inside the github actions

filip326 commented 7 months ago

@oglimmer you've got most experince with gh actions and this stuff

chfuchte commented 7 months ago

its completly possible. its just a question, if its required after a year of development..

chfuchte commented 7 months ago

and since the actions've used npm since they exist, I don't get why you used yarn the whole time ;)

filip326 commented 7 months ago

and since the actions've used npm since they exist, I don't get why you used yarn the whole time ;)

Because @Mickhat asked me to

chfuchte commented 7 months ago

aha (oder auf Deutsch: immer schön die Schuld auf andere schieben ;))

filip326 commented 7 months ago

image (auf Deutsch: Beweis)

chfuchte commented 7 months ago

ja ok

EinsKatze commented 7 months ago

npm usually gets installed when installing NodeJS. (So why install & switch to yarn?) btw what would be the benefit of using yarn instead of npm?

Mickhat commented 7 months ago

habbit i guess

chfuchte commented 1 month ago

habbit i guess

damit wäre wohl alles gesagt