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

Removes magic stuff adjacent to gun/magic #2877

Open
wants to merge 102 commits into
base: master
Choose a base branch
from

Conversation

FalloutFalcon
Copy link
Contributor

@FalloutFalcon FalloutFalcon commented Apr 4, 2024

About The Pull Request

This was MEANT to be a pr were i was tackling gun cruft but magic accounts for a large portion of the line count.

Pulls from #2664 (minus the actual gun crafting) and #2702

Why It's Good For The Game

Cruft

Changelog

🆑
del: Removed some magic cruft
/:cl:

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Apr 11, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@FalloutFalcon FalloutFalcon marked this pull request as ready for review April 16, 2024 12:36
@FalloutFalcon FalloutFalcon requested a review from a team as a code owner April 16, 2024 12:36
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Apr 18, 2024
FalloutFalcon and others added 7 commits April 18, 2024 13:23
Signed-off-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
Signed-off-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
oops some how readded it

Signed-off-by: FalloutFalcon <86381784+FalloutFalcon@users.noreply.github.com>
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Apr 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Apr 23, 2024
@FalloutFalcon
Copy link
Contributor Author

linters failing and i cant tell why.

@FalloutFalcon
Copy link
Contributor Author

wtf is causing this
html/changelogs/example.yml: OK
/home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2944.yml: Invalid prefix tweak
/home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2924.yml: Invalid prefix tweak
Reading changelogs...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2944.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2944.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2947.yml...
Added 2 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2947.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2922.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2922.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2938.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2938.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2946.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2946.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2924.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2924.yml (delete-after set)...
Reading /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2935.yml...
Added 1 new changelog entries.
Deleting /home/runner/work/Shiptest/Shiptest/html/changelogs/AutoChangeLog-pr-2935.yml (delete-after set)...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. DME Edit Map Change Tile placing is hard. Thank you for your service.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant