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

Sample Images for FlexBox5, SideDoorHousing,Tetris #632

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

littleroblet
Copy link
Contributor

@littleroblet littleroblet commented Feb 27, 2024

This adds Samples Images for FlexBox5, SideDoorHousing, and Tetris
There is a second image for SideDoorHousing (I can add this to the generator as a second image in a separate commit)

included samples.sha256

Please let me know if I have done anything incorrectly. It was my first time committing to this project.

@florianfesti
Copy link
Owner

This looks pretty good already. If you want you can put the main images through https://tinypng.com/ as was done with most other pictures and regenerate the thumbnails after wards. But I can also just merge now and to that later myself.

Should probably add that step to the CONSTRIBUTING file...

@littleroblet
Copy link
Contributor Author

Up to you but I can give it a go tomorrow. :)

@littleroblet littleroblet marked this pull request as draft February 27, 2024 21:49
@littleroblet
Copy link
Contributor Author

@florianfesti I have put the images through tinypng.com and added a new commit.

Please let me know if my Pull Request is done wrong. I am still learning to use GIT/Github properly. One of the reasons I did not just upload the images to the ticket.

@littleroblet littleroblet marked this pull request as ready for review February 27, 2024 22:17
@florianfesti florianfesti merged commit 0fe9d0e into florianfesti:master Feb 28, 2024
@florianfesti
Copy link
Owner

Looks good! merged!

@littleroblet littleroblet deleted the SampleImages branch February 29, 2024 22:28
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