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

Revert "[BombasticPerks] A new playstyle perk: Enchanting Master" #72217

Merged
merged 2 commits into from
Mar 9, 2024

Conversation

kevingranade
Copy link
Member

Summary

None

Reverts #71702

This was directly copying from shattered pixel dungeon without attribution.

The names aren't necessarily a problem, but the descriptions must be original.

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. NPC / Factions NPCs, AI, Speech, Factions, Ownership Translation I18n [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding [C++] Changes (can be) made in C++. Previously named `Code` [Markdown] Markdown issues and PRs [Python] Code made in Python EOC: Effects On Condition Anything concerning Effects On Condition Mods: Bombastic Perks labels Mar 7, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Auto-requesting reviews from non-collaborators: @bombasticSlacks

@Inglonias

This comment was marked as off-topic.

@I-am-Erk I-am-Erk mentioned this pull request Mar 8, 2024
1 task
@kevingranade kevingranade force-pushed the revert-71702-CUSTOM_ITEM_SUFFIX branch from 790c7bb to 2a9b20d Compare March 8, 2024 00:17
@Qrox
Copy link
Contributor

Qrox commented Mar 8, 2024

I wonder if there are other texts with copyright issues like this. After all these are hard to spot and thus may have slipped in more than once. This also reminds me of seeing a description which looked like it was directly copied from Wikipedia (which is using CC-BY-SA 4.0 so we can probably just attribute to it?)

@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Mar 8, 2024
@kevingranade
Copy link
Member Author

It is something we need to watch out for yes.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Mar 8, 2024
@kevingranade kevingranade merged commit 927c4ba into master Mar 9, 2024
19 of 24 checks passed
@kevingranade kevingranade deleted the revert-71702-CUSTOM_ITEM_SUFFIX branch March 9, 2024 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` <Documentation> Design documents, internal info, guides and help. EOC: Effects On Condition Anything concerning Effects On Condition [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions [Markdown] Markdown issues and PRs Mods: Bombastic Perks Mods Issues related to mods or modding NPC / Factions NPCs, AI, Speech, Factions, Ownership [Python] Code made in Python Translation I18n
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants