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 credits tab #3907

Merged
merged 1 commit into from May 9, 2016
Merged

Update credits tab #3907

merged 1 commit into from May 9, 2016

Conversation

est31
Copy link
Contributor

@est31 est31 commented Mar 24, 2016

  • Move resigned core devs BlockMen and Zeno to the previous core devs section
  • Add two new contributors

@duane-r @sofar I have added you to the list of contributors, do you want to change something about your appearance?

This PR should be merged shortly before the release so that we can update it if some new contributors get added.

@kilbith
Copy link
Contributor

kilbith commented Mar 24, 2016

@sofar is going to be core-dev soon. It's not his place to be in contributors between 0.4.14 and 0.4.15 (rather in the rank above).

@est31
Copy link
Contributor Author

est31 commented Mar 24, 2016

@kilbith i was in contributors too before I became core dev. Whether he becomes core dev or whether he wants to become such is a different matter. I just looked at git shortlog 0.4.13..HEAD -n and saw sofar with multiple big (non-translation) contributions.

@est31 est31 added this to the 0.4.14 milestone Mar 24, 2016
@sofar
Copy link
Contributor

sofar commented Mar 24, 2016

Contributors is fine. I was asked and have told several folks that I'd like to do one release as contributor.

@paramat
Copy link
Contributor

paramat commented Mar 25, 2016

Next time you update this, please could you change my full name to 'Matt Gregory'? i used to spell it with 1 't' but prefer 2 now.
I agree sofar should not be 'core dev' before he is a core dev.

@est31
Copy link
Contributor Author

est31 commented Mar 25, 2016

@paramat done

@kilbith
Copy link
Contributor

kilbith commented Mar 25, 2016

The proposed dedication of the 0.4.14 release should be done in another PR.

Why ? It fits the "Update credits tab" scope and the honouring is more expected from a core-dev.

@est31
Copy link
Contributor Author

est31 commented Mar 25, 2016

Okay then, thought you wanted to provide some text. I have added a proposal, we could include the same text in polish below, @red-001 can you translate it and reply it in this PR? Then I can include it into the PR.

@@ -80,6 +82,11 @@ tab_credits = {
"tablecolumns[color;text]" ..
"tableoptions[background=#00000000;highlight=#00000000;border=false]" ..
"table[3.5,-0.25;8.5,5.8;list_credits;" ..
"#FFFF00," .. fgettext("Dedication of the current release") .. ",," ..
"The " .. core.get_version() .. " release is dedicated to the memory of" .. ",," ..
Copy link
Contributor

Choose a reason for hiding this comment

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

The version getter is unnecessary since only 0.4.14 is dedicated to RBA.

@kilbith
Copy link
Contributor

kilbith commented Mar 25, 2016

English-only is fine so that everyone understand it.

@est31
Copy link
Contributor Author

est31 commented Mar 25, 2016

its just an idea to have it bilingual

"#FFFF00," .. "Dedication of the current release" .. ",," ..
"The 0.4.14 release is dedicated to the memory of" .. ",," ..
"Minetest developer Maciej Kasatkin (RealBadAngel)" .. ",," ..
"who died on March 23 2016." .. ",," ..
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it was the 24th?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for spotting, it indeed is march 24.

"Maciej Kasatkin (RealBadAngel) <maciej.kasatkin@o2.pl>",
"Lisa Milne (darkrose) <lisa@ltmnet.com>",
"proller",
"Craig Robbins (Zeno)",
Copy link
Contributor

Choose a reason for hiding this comment

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

Zeno actually quit? Or is he currently inactive?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kwolekr
Copy link
Contributor

kwolekr commented Mar 31, 2016

gregorycu is actively contributing, and he could be made a core developer soon as well, FWIW.

@kilbith
Copy link
Contributor

kilbith commented Apr 1, 2016

@est31 est31 merged commit 95446f4 into minetest:master May 9, 2016
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

5 participants