Skip to content

Update CI workflow - #55

Merged
merwaaan merged 7 commits into
merwaaan:mainfrom
mgi388:ci.yaml
Oct 7, 2025
Merged

Update CI workflow#55
merwaaan merged 7 commits into
merwaaan:mainfrom
mgi388:ci.yaml

Conversation

@mgi388

@mgi388 mgi388 commented Oct 4, 2025

Copy link
Copy Markdown
Contributor

Alternative to #54 (comment).

I first copied the file from https://github.com/TheBevyFlock/bevy_new_2d/blob/9fd68f61dd9c31f8a73d033fdfc9e0a33dc23a88/.github/workflows/ci.yaml#L1-L186 and deleted check-web but then decided that because that's a game template maybe it's not suitable. I know that benfrankel is a contributor to that repo, so I went and looked at one of his libraries: https://github.com/benfrankel/pyri_tooltip.

I then updated the PR here, instead copying pyri_tooltip's ci.yaml and going from there to make it pass CI.

The PR commit history shows the changes I made if that's useful to see.

@mgi388
mgi388 marked this pull request as draft October 4, 2025 08:15
@mgi388 mgi388 changed the title Use ci.yaml from bevy_new_2d Update CI workflow Oct 4, 2025
@mgi388
mgi388 marked this pull request as ready for review October 4, 2025 08:27
@mgi388

mgi388 commented Oct 4, 2025

Copy link
Copy Markdown
Contributor Author

@merwaaan this CI update PR passes now, ready for review when you get a chance.

@merwaaan

merwaaan commented Oct 7, 2025

Copy link
Copy Markdown
Owner

Thanks again @mgi388 :)

@merwaaan
merwaaan merged commit c26ba68 into merwaaan:main Oct 7, 2025
4 checks passed
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