Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed up the files #13

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

red-socks294
Copy link

Okay. I have updated some of the code and made the code easier to understand, according to PyCharm IDE.

@mandar1jn
Copy link

Have you tried to run your code? I checked your repository and it doesn’t contain a data folder. This should not be merged.

@mandar1jn
Copy link

Also: why did you first delete everything?

@red-socks294
Copy link
Author

red-socks294 commented Mar 22, 2021 via email

@mandar1jn
Copy link

@fel1x0u what?

@red-socks294
Copy link
Author

red-socks294 commented Mar 23, 2021 via email

@mandar1jn
Copy link

I get that, but why did you delete the file? You could have just forked it and then cloned the repo. I don’t see the need to delete all files. Also: what happened to the data folder?

@red-socks294
Copy link
Author

red-socks294 commented Mar 24, 2021 via email

@ghost
Copy link

ghost commented Apr 14, 2021

On Mon, Mar 22, 2021 at 6:35 AM mandar1jn @.***> wrote: Also: why did you first delete everything? ?You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe. Because, I edited it on my PC. I used an external IDE

If PolyMars accepts this the game would literally break, I think you should update main close this PR and create a new one.

@ghost
Copy link

ghost commented Apr 14, 2021

If there is no data folder the game won't have audio, images and nothing of that sort. It would probably give anyone who tries playing 1000 errors.

@mandar1jn
Copy link

Indeed

@mandar1jn
Copy link

On Wed, Mar 24, 2021 at 1:53 AM mandar1jn @.***> wrote: I get that, but why did you delete the file? You could have just forked it and then cloned the repo. I don?t see the need to delete all files. Also: what happened to the data folder? ?You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe. I forgot to upload it. Let me fix that

I get that, but that is still no answer to my question. Why didn’t you just clone the repository?

@red-socks294
Copy link
Author

red-socks294 commented Apr 14, 2021 via email

@red-socks294
Copy link
Author

okay done

@mandar1jn
Copy link

@fel1x0u I still want an answer. Why did you delete all the files instead of cloning the repo?

@firefish111
Copy link

firefish111 commented Apr 15, 2021 via email

@firefish111
Copy link

brrrrrr does markdown even work ééééééééééééééééééééééééééééééééééééééé

@mandar1jn
Copy link

@firefish111 I get what you mean, but the thing I am wondering is why he deleted all the files first.

@red-socks294
Copy link
Author

red-socks294 commented Apr 15, 2021 via email

@firefish111
Copy link

@fel1x0u You could've done git checkout then git push... or at least i think that's how it works

@red-socks294
Copy link
Author

red-socks294 commented Apr 15, 2021 via email

@firefish111
Copy link

@fel1x0u Well... how did you make a pull request if you don't use git?

@red-socks294
Copy link
Author

red-socks294 commented Apr 16, 2021 via email

@firefish111
Copy link

@fel1x0u GitHub is a git repository hosting platform...

@mandar1jn
Copy link

In my opinion #7 should be merged instead of this. It is cleaner and doesn’t have a lot of mentioned file changes that aren’t needed

@firefish111
Copy link

Is this pull dead? I was cleaning my emails and came across this.

@mandar1jn
Copy link

Is this pull dead? I was cleaning my emails and came across this.

Polymarsdev kinda just doesn’t check github

@firefish111
Copy link

@mandar1jn i'd know, it took around a month to reply to the bean on the left plus on the right thingo

@mandar1jn
Copy link

@firefish111 I have pull requests on his repositories that are almost a year old

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.

3 participants