Skip to content

Voronoi game of life submission #1624

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

Closed
wants to merge 2 commits into from
Closed

Voronoi game of life submission #1624

wants to merge 2 commits into from

Conversation

okg21
Copy link

@okg21 okg21 commented Jun 16, 2024

Voronoi game of life showcase submission

Copy link

netlify bot commented Jun 16, 2024

Deploy Preview for codingtrain ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 5efac25
🔍 Latest deploy log https://app.netlify.com/sites/codingtrain/deploys/666eef04a739a700073951c9
😎 Deploy Preview https://deploy-preview-1624--codingtrain.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@fturmel fturmel added the showcase Passenger Showcase Submission label Jun 16, 2024
@shiffman
Copy link
Member

shiffman commented Jul 3, 2024

Ah, whoops, I think this was already merged in #1622, can you confirm?

@okg21
Copy link
Author

okg21 commented Jul 3, 2024

Hey, yes, the only difference is this one goes directly to a demo. If it is OK, I prefer this one. But it's OK anyway. Thank you.

@shiffman
Copy link
Member

shiffman commented Jul 4, 2024

Hey, yes, the only difference is this one goes directly to a demo. If it is OK, I prefer this one. But it's OK anyway. Thank you.

Ah, that makes sense! Since I already merged #1622, would you like to adjust this branch (or open a new pull request) that changes the link in the file that is now in main?

@okg21
Copy link
Author

okg21 commented Jul 4, 2024

Hey, yes, the only difference is this one goes directly to a demo. If it is OK, I prefer this one. But it's OK anyway. Thank you.

Ah, that makes sense! Since I already merged #1622, would you like to adjust this branch (or open a new pull request) that changes the link in the file that is now in main?

Thanks, I will open a pull request that updates the link.

@fturmel fturmel mentioned this pull request Jul 15, 2024
@fturmel
Copy link
Collaborator

fturmel commented Jul 15, 2024

@okg21 Hope you don't mind, I opened a PR to close this issue. See #1665

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting update showcase Passenger Showcase Submission
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants