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

Change explosion method #77

Merged
merged 3 commits into from
Mar 12, 2024
Merged

Conversation

Rxann
Copy link
Contributor

@Rxann Rxann commented Feb 28, 2024

Change the explosion method so you cannot see that the user blew someone up, instead it shows they just died. Also changed the directory from HellBase to HorseMenu because well this is HorseMenu.

Copy link
Member

@Yimura Yimura left a comment

Choose a reason for hiding this comment

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

You need to update the files under .github/workflows as they rely on the TemplateHellBase project and dll name.

@Rxann
Copy link
Contributor Author

Rxann commented Mar 9, 2024

You need to update the files under .github/workflows as they rely on the TemplateHellBase project and dll name.

I just looked in the files and it seems that everything is named HorseMenu properly? I have only worked with the workflows a couple of times so I could be missing something.

@Yimura
Copy link
Member

Yimura commented Mar 9, 2024

Let's see if it works.

Copy link

github-actions bot commented Mar 9, 2024

Download the artifacts for this pull request:

@Rxann
Copy link
Contributor Author

Rxann commented Mar 9, 2024

Let's see if it works.

It passed

@Rxann Rxann changed the title Change explosion method + directory change Change explosion method Mar 9, 2024
@Rxann
Copy link
Contributor Author

Rxann commented Mar 9, 2024

I moved the directory change the the pr that fixes the crashes so that why i updated the title

@Yimura Yimura merged commit 99190d3 into YimMenu:master Mar 12, 2024
1 check passed
@Rxann Rxann deleted the change-explosion-method branch March 12, 2024 14:35
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