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

adds mobile game dev fridays group #257

Merged
merged 6 commits into from
Sep 13, 2024

Conversation

JimLynchCodes
Copy link
Contributor

👾

@mightyiam
Copy link
Member

Thank you @JimLynchCodes for making the effort.

Here's an error. Ideally, it would be more useful.

https://github.com/mobusoperandi/website/actions/runs/10751871553/job/29819562238?pr=257#step:8:518

hrm... sorry, I don't get it. You have any thoughts perhaps?

@JimLynchCodes
Copy link
Contributor Author

Thanks @mightyiam

I think I was using an invalid value for the "status"... or possible the colors. But I just changed both of those things, and it seems to build fine for me locally so I'm hoping that it will go fine on ci now. I don't see how I can get to the run for this commit though.

@mightyiam
Copy link
Member

It's set up in a way that CI runs must be approved individually. Which I just did. So let's see.

@JimLynchCodes
Copy link
Contributor Author

JimLynchCodes commented Sep 12, 2024

@mightyiam I think the build completed successfully, right?

Can you merge pls so I can start telling people about the meeting tomorrow :)

@mightyiam
Copy link
Member

Sorry. Can't seem to do on the phone. Tomorrow then.

Copy link
Member

@mightyiam mightyiam left a comment

Choose a reason for hiding this comment

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

I'm not sure what's going on with the deployment not finding some secret. I'm sure I had this set up correctly at some point. Thinking it might be waiting for an approving review (long shot).

@mightyiam mightyiam merged commit ee518f0 into mobusoperandi:master Sep 13, 2024
2 of 3 checks passed
@JimLynchCodes
Copy link
Contributor Author

Woo, I can see it now! I think I did the date math incorrectly though bc it's showing up at the wrong time of day... 😅

I'll fix it and open another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: done
Development

Successfully merging this pull request may close these issues.

2 participants