Skip to content

UNO game#92

Open
mayank-gariya wants to merge 9 commits into
Grow-with-Open-Source:mainfrom
mayank-gariya:main
Open

UNO game#92
mayank-gariya wants to merge 9 commits into
Grow-with-Open-Source:mainfrom
mayank-gariya:main

Conversation

@mayank-gariya

Copy link
Copy Markdown

No description provided.

@github-actions

Copy link
Copy Markdown

👋 @mayank-gariya
Thank you for raising your pull request.
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.

@mayank-gariya mayank-gariya left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

play uno build by you

@iamwatchdogs iamwatchdogs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mayank-gariya, please make the following changes,

  • Remove any media files (like PNG files)
  • Remove unnecessary empty text files (like uno_details.txt)
  • It seems like logic.py file is not being used by any of the project file. Please remove it or refactor it to somehow use the file.

Please don't create a new commit to implement the requested changes. Instead, amend the existing commit and force push the changes to the PR branch.

@mayank-gariya

Copy link
Copy Markdown
Author

Hi
Thank you for the review.

I've addressed the requested changes:

  • Removed all unnecessary media files (PNG/images) from the repository.
  • Removed the unused text file (uno_details.txt).
  • Removed the unused logic.py file, as it was not being referenced by any part of the project.

I've also amended the existing commit and force-pushed the updated changes to the PR branch instead of creating a new commit, as requested.

after folder structure :
image

Please let me know if there's anything else that should be updated.

Thank you!

@github-actions

Copy link
Copy Markdown

👋 @mayank-gariya This PR is closed. If you think there's been a mistake, please contact the maintainer @iamwatchdogs.

@mayank-gariya

Copy link
Copy Markdown
Author

hey can i get the process ! is it closed or opened

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants