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

Renamed nuke to target to be considerate of nuclear attack survivors and their family #370

Merged
merged 2 commits into from
May 9, 2022

Conversation

Wallby
Copy link
Contributor

@Wallby Wallby commented May 9, 2022

I.e. in response to #367..
.. renamed "nuke" everywhere w. target
.. removed radioactive.png and replaced w. target.png (made w. paint dot net)

I have not ran tests nor tried to compile, I do not have any of the tools installed on my computer.

@Klocman
Copy link
Owner

Klocman commented May 9, 2022

Seems fine on a glance, please change the PR to ready for review once it's ready for testing.

@Wallby
Copy link
Contributor Author

Wallby commented May 9, 2022

@Klocman I think it is ready for testing (i.e. I did a scan for any occurances of radioactive and nuke and fixed 'm all up) though do not plan on doing testing myself. Can I still change this to ready for review?

@Klocman
Copy link
Owner

Klocman commented May 9, 2022

@Klocman I think it is ready for testing (i.e. I did a scan for any occurances of radioactive and nuke and fixed 'm all up) though do not plan on doing testing myself. Can I still change this to ready for review?

Yes, if you don't plan to change anything else then you should set it as ready. Draft PRs are generally ignored until they are ready.

@Klocman Klocman marked this pull request as ready for review May 9, 2022 19:17
Copy link
Owner

@Klocman Klocman left a comment

Choose a reason for hiding this comment

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

After the fix everything works fine.

@Klocman Klocman merged commit 3ec0bf8 into Klocman:master May 9, 2022
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.

None yet

2 participants