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

Update Credit.txt of MSX #1366

Closed
wants to merge 1 commit into from
Closed

Conversation

chaosvolt
Copy link

Summary

MSX "Update credit.txt of MSX"

Content of the change

Credits file of MSX was missing a contributor listed. Literally my first contribution to DDA was CleverRaven/Cataclysm-DDA#11707, which was when the Mshock set was developed into MShock Modded, which was used as the base by Xotto (source: https://discourse.cataclysmdda.org/t/20170216-big-sprite-update-mshock32xottoplus-tileset-oversprite-tileset/11301).

Testing

N/A

Additional information

@Fris0uman
Copy link
Collaborator

What sprite did you add? We can maybe give credit directly for those sprites

@chaosvolt
Copy link
Author

What sprite did you add? We can maybe give credit directly for those sprites

There were a full 33 PRs pertaining to MShock Modded: https://github.com/CleverRaven/Cataclysm-DDA/pulls?q=is%3Apr+mshock+author%3Achaosvolt That's kinda a lot to search through, and probably why listing prior tileset authors and major contributors in a single credit file seems to be the normal practice.

@Fris0uman
Copy link
Collaborator

Since you're here it would be very helpful if you could actually identify your contributions, it would allow for a nicer tracking of the credits.

@I-am-Erk
Copy link
Owner

I-am-Erk commented Jun 6, 2022

CV, we'd be more than happy to credit you. It is kind of the main reason we started this project. You don't need to dramatically complain in other communities about it, Fris0 is trying to help you get the same type of direct credit every other artist enjoys in the project, instead of a vague wave towards "I guess CV did some stuff but nobody knows what". It's meant to be respectful, not hostile.

@chaosvolt
Copy link
Author

It would take a fair amount of sifting through the PRs to track all the sprites I'd added, then double-checking which ones are still present in the tileset. I'd figured that just updating the credit text was more than sufficient.

@I-am-Erk
Copy link
Owner

I-am-Erk commented Jun 6, 2022

It is, this isn't a question of sufficiency or asking you to "prove it", it is an offer of equality to other artists whose work have been added.

@chaosvolt
Copy link
Author

If I can find time I can try to look through later on, thank you.

@Fris0uman
Copy link
Collaborator

The solution provided by #1368 obsoletes this PR. But the list of sprite you've made is still welcome and we'll add attribution on those sprites as soon as you provide it. Feel to PR it by puting the sprite in a folder named like the sprite and adding credits.txt file to that folder with your name in it. Or opening an issue with the list of sprites and I'll take care of the PR.

@Fris0uman Fris0uman closed this Jun 6, 2022
@chaosvolt
Copy link
Author

PR linked does not actually include the change this PR added.

@chaosvolt chaosvolt deleted the msx-update branch June 6, 2022 17:01
@Fris0uman
Copy link
Collaborator

It links to the whole history of the tileset when it lived in the dda repo, therefor it does give credit to anyone that PRed something to that tileset at the time.

@I-am-Erk
Copy link
Owner

I-am-Erk commented Jun 6, 2022

It includes all the PRs you just linked, with your name and specific contributions.

https://github.com/CleverRaven/Cataclysm-DDA/commits/60ff294b9613ae6cddcb250e84f7e66921e5f3ad?after=60ff294b9613ae6cddcb250e84f7e66921e5f3ad+244&branch=60ff294b9613ae6cddcb250e84f7e66921e5f3ad&path%5B%5D=gfx&qualified_name=60ff294b9613ae6cddcb250e84f7e66921e5f3ad eg

It's what we likely should have been doing in the first place as it gives much better credit information.

@chaosvolt
Copy link
Author

Guess that works then.

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

3 participants