Gamecord icon indicating copy to clipboard operation
Gamecord copied to clipboard

Add guess the number game

Open retrouser955 opened this issue 3 years ago • 7 comments

Changes

describe what changes this PR includes and explain why are they needed

Screenshots

If applicable, add screenshots to explain the change.

Status

  • [x] These changes have been tested and formatted properly.
  • [x] These changes also include the change in documentation accordingly.
  • [ ] This PR introduces some Breaking changes

retrouser955 avatar Nov 20 '22 12:11 retrouser955

👍🏻 coolio

gspxrk avatar Nov 20 '22 14:11 gspxrk

One thing caught my attention, why is a yarn.lock file added?

ChrisSch-dev avatar Nov 25 '22 02:11 ChrisSch-dev

One thing caught my attention, why is a yarn.lock file added?

Tbh, I have no idea 😂. I always add it to all my projects. I can remove it if Aniket say so.

retrouser955 avatar Nov 25 '22 08:11 retrouser955

This pull request probably won't be merged until the new bug fix update is published.

gspxrk avatar Nov 26 '22 09:11 gspxrk

Yes, the owner of the repo (Aniket) have told me in discord about this.

retrouser955 avatar Nov 26 '22 09:11 retrouser955

One thing caught my attention, why is a yarn.lock file added?

Tbh, I have no idea 😂. I always add it to all my projects. I can remove it if Aniket say so.

You should probably remove the yarn.lock then. It isn't necessary here, and it may cause some issue.

ChrisSch-dev avatar Nov 26 '22 11:11 ChrisSch-dev

I just read. yarn.lock provide yarn on the information about dependencies of the project and tell yarn how to manage the project. The post states that, if you do not push lock files (package-lock.json, yarn.lock), it could cause issues on other machines. Thus, I will be keeping this file for now and I will remove it when the pull request is getting close to merging.

retrouser955 avatar Nov 26 '22 11:11 retrouser955

Oh 😅 I didn't realise the pull request was open haha. I was just clearing out repos that I am inactive on.

retrouser955 avatar Jun 09 '24 13:06 retrouser955