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

silly dog eats things #102

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

silly dog eats things #102

wants to merge 23 commits into from

Conversation

NiliUnger
Copy link

slack username - Nili
basically a game where a copy of my real dog eats things he should (or shouldn't) be eating in order to gain points to fill up a hunger bar

slack username - Nili
Copy link

vercel bot commented Feb 4, 2025

@NiliUnger is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

@acornitum acornitum added the needs review needs acon to check over it , ,,,, label Feb 4, 2025
added some sprite changes and added idle animation + for jump
@acornitum
Copy link
Contributor

Hey - could you put your code in a folder in /pets instead? Thanks! Will check after it :D

Feel free to start a new PR or update this one!

@acornitum acornitum removed the needs review needs acon to check over it , ,,,, label Feb 6, 2025
i didn't upload the code to the right place at first lol - nili
@NiliUnger
Copy link
Author

@acornitum i think i fixed it? im new to github so i may have done it wrong but i think the code is in the right place now

@acornitum
Copy link
Contributor

you've moved your code into /pets! Could you create a folder in /pets and move all your code + art into it? (call the folder silly_dog or something! use / to create the folder)

i think this is it? i hope this works
@NiliUnger
Copy link
Author

okay now i got it
yay!

@acornitum
Copy link
Contributor

nice! you still got a few more files you need to add into the folder haha

image

@acornitum
Copy link
Contributor

actually, i think you already have them in the folder - in that case, delete the file from their original position!

@NiliUnger
Copy link
Author

that should be everything - lmk if theres anything else i need to do!

@acornitum
Copy link
Contributor

put your hackapet_code.py file in the folder too!

@NiliUnger
Copy link
Author

should be there!

@acornitum acornitum added the needs review needs acon to check over it , ,,,, label Feb 6, 2025
@acornitum
Copy link
Contributor

You have the wrong file place linked here!
image

@acornitum acornitum removed the needs review needs acon to check over it , ,,,, label Feb 8, 2025
@NiliUnger
Copy link
Author

that's so weird it should be the right file... let me see what i can do, and i am SO sorry for all of the issues

@NiliUnger
Copy link
Author

okay im resaving the bmp file and changing the code and we'll see what that does

@NiliUnger
Copy link
Author

OKAY HERE'S THE SUMMARY:
A. i had to reupload every single art file and change the code some. the good news? the game's working a lot more smoothly
something i have noticed: i did the stupid thing of including some spaces in my bmp files! so imma send a screenshot of the bmp files correct names because my vscode likes to put underscores in them (i am so sorry for all of this). also im changing the art files to make sure they have the correct paths
B. im reuploading the code with the changes made. i had to change the path for literally every image so i hope this works but it'll be okay if it's not it's not the worst thing to change
C. THE SCREENSHOTS:
Screenshot 2025-02-09 114131
Screenshot 2025-02-09 114146
I AM SO SORRY ACON THIS HAS BEEN A NIGHTMARE IM SURE

god please let this work
okay why do things keep on getting pulled out... this should be it?
@acornitum acornitum added the needs review needs acon to check over it , ,,,, label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review needs acon to check over it , ,,,,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants