From 06610c9f86b81abf29040b723f3ec07d61a7a06a Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 11:54:35 -0800 Subject: [PATCH 1/8] Create MAINLINE_MODS.md --- doc/MAINLINE_MODS.md | 57 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 57 insertions(+) create mode 100644 doc/MAINLINE_MODS.md diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md new file mode 100644 index 0000000000000..f4ff4593a913d --- /dev/null +++ b/doc/MAINLINE_MODS.md @@ -0,0 +1,57 @@ +# Getting Mods Into The Repository + +Cataclysm: Dark Days Ahead is not only moddable, but ships with a number of mods available for users to select from even without having to obtain mods from a third party source. + +## Why have mainlined mods? + +The benefits of having mods mainlined include: + +* Visibility to a large number of users and potential contributors +* The ability to use the CleverRaven issue tracker to report problems, as well as manage pull requests for adding to or fixing mods in the repository +* Mods in mainline that demonstrate usage of a game feature which exists only for mods (i.e. not used by the core DDA game itself) help ensure that feature stays present in the game. + +There are however things that might be expected but are not guaranteed: + +* Guarantee of maintenance of content -- core devs are mostly focused on the main game itself; although some might pick up mods as a side task, people who continue working on the core game tend to like the feel of the core game. +* Guarantee of maintenance of used features -- over the course of development it may become impossible to keep a feature, due to performance needs or maintenance burden. +* Guarantee of sole authorship -- once in the repository, the mod is considered community content. While the developers might defer to an active maintainer of a mod, if they become inactive anyone else can step forward. + +## What kind of mods can be in the repository as mainlined mods? + +There are three primary categories of mods: + +* Content mods, which provide some kind of distinct experience from the core Dark Days Ahead game. This could be a change to lore, a change to a specific part of gameplay. +* User Experience (UX) mods, which alter the look and feel of the game interface itself. +* Development mods, which aren't "mods" in the typical sense but are instead there to ease the transition between "incomplete feature" and "incomplete feature", when a feature in the core game is sufficiently incomplete that the developers believe it needs to be optional to minimize disruption to players. + +## What is necessary for a mod to be included in the repository? + +The most crucial criteria for a mod to be in the CleverRaven repository is that it has someone acting as a maintainer. This ensures that there is someone who is keeping an eye out for possible problems with the mod, and helping steer its development so that it continues to develop in accordance with its design purpose. + +Further, there are additional criteria: + +* Content mods need to be providing some kind of curated experience. Either they introduce a set of locations, encounters, equipment, or progression mechanisms that do not fit the style of the core game, or they in some fashion modify the game according to some well-defined concept. +* * Mods which do not qualify as curated experiences: Grab bag mods with no defined purpose (i.e. adding a bunch of randomly guns), "settings" mods that just turn off a possibly undesired but working feature (acid ants). +* Development mods need to have an associated work-in-progress feature that they exist to mitigate. + +## What are the responsibilities of a mod maintainer? + +`TBD` + +## When do mods get removed? + +Mods that have no maintainer or have proven to be consistently some type of maintenance burden are potentially subject to removal. Additionally, mods which had a purpose which is no longer relevant (development mods whose accompanying feature has been finished) will generally be removed once they're no longer deemed necessary. + +Whenever a stable release occurs, mods which no longer qualify for inclusion will be marked 'obsolete'. This doesn't remove them from the repository, but it does prevent it from being used in new worlds (barring additional effort on the part of the player). + +## Why do mods get removed? + +While it may be counterintuitive, simply having a mod in the repository increases maintenance burden for the team, even for the people who don't use the mods itself. + +Further, mods which are in mainline but are not working correctly for whatever reason reflect poorly on the project on the whole. While it might seem 'easy' to fix a mod, the development team is mostly focused on the core game itself. This is why mod maintainers are necessary. + +## How can mods be rescued from removal? + +If the mod otherwise meets inclusion criteria but lacks a maintainer, it's as simple as having someone else step forward as the new maintainer. + +Otherwise, it needs to either be made to meet the criteria, or it simply isn't going to be staying in mainline. From dba99ff33edb0eca03ae9860f3345785aa3c289e Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 12:09:24 -0800 Subject: [PATCH 2/8] Update doc/MAINLINE_MODS.md Co-Authored-By: Curtis Merrill --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index f4ff4593a913d..82e17f0863b8d 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -31,7 +31,7 @@ The most crucial criteria for a mod to be in the CleverRaven repository is that Further, there are additional criteria: * Content mods need to be providing some kind of curated experience. Either they introduce a set of locations, encounters, equipment, or progression mechanisms that do not fit the style of the core game, or they in some fashion modify the game according to some well-defined concept. -* * Mods which do not qualify as curated experiences: Grab bag mods with no defined purpose (i.e. adding a bunch of randomly guns), "settings" mods that just turn off a possibly undesired but working feature (acid ants). +* * Mods which do not qualify as curated experiences: Grab bag mods with no defined purpose (i.e. adding a bunch of random guns), "settings" mods that just turn off a possibly undesired but working feature (acid ants). * Development mods need to have an associated work-in-progress feature that they exist to mitigate. ## What are the responsibilities of a mod maintainer? From c9a4e57246e35f8db8444b8cf27d0a0c296e29a9 Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 12:09:37 -0800 Subject: [PATCH 3/8] Update doc/MAINLINE_MODS.md Co-Authored-By: Fris0uman <41293484+Fris0uman@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index 82e17f0863b8d..bff59f116bbc9 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -22,7 +22,7 @@ There are three primary categories of mods: * Content mods, which provide some kind of distinct experience from the core Dark Days Ahead game. This could be a change to lore, a change to a specific part of gameplay. * User Experience (UX) mods, which alter the look and feel of the game interface itself. -* Development mods, which aren't "mods" in the typical sense but are instead there to ease the transition between "incomplete feature" and "incomplete feature", when a feature in the core game is sufficiently incomplete that the developers believe it needs to be optional to minimize disruption to players. +* Development mods, which aren't "mods" in the typical sense but are instead there to ease the transition between "incomplete feature" and "complete feature", when a feature in the core game is sufficiently incomplete that the developers believe it needs to be optional to minimize disruption to players. ## What is necessary for a mod to be included in the repository? From c115ababd9206c497313591d7c612fb277e26351 Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 13:48:50 -0800 Subject: [PATCH 4/8] Update doc/MAINLINE_MODS.md Co-Authored-By: davidpwbrown <39344466+davidpwbrown@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index bff59f116bbc9..5a028b69e5819 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -28,7 +28,7 @@ There are three primary categories of mods: The most crucial criteria for a mod to be in the CleverRaven repository is that it has someone acting as a maintainer. This ensures that there is someone who is keeping an eye out for possible problems with the mod, and helping steer its development so that it continues to develop in accordance with its design purpose. -Further, there are additional criteria: +Furthermore, there are additional criteria: * Content mods need to be providing some kind of curated experience. Either they introduce a set of locations, encounters, equipment, or progression mechanisms that do not fit the style of the core game, or they in some fashion modify the game according to some well-defined concept. * * Mods which do not qualify as curated experiences: Grab bag mods with no defined purpose (i.e. adding a bunch of random guns), "settings" mods that just turn off a possibly undesired but working feature (acid ants). From add9d629a48a1d54ce22a994b3af7557448d9cef Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 13:48:59 -0800 Subject: [PATCH 5/8] Update doc/MAINLINE_MODS.md Co-Authored-By: davidpwbrown <39344466+davidpwbrown@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index 5a028b69e5819..53d8a58140df2 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -48,7 +48,7 @@ Whenever a stable release occurs, mods which no longer qualify for inclusion wil While it may be counterintuitive, simply having a mod in the repository increases maintenance burden for the team, even for the people who don't use the mods itself. -Further, mods which are in mainline but are not working correctly for whatever reason reflect poorly on the project on the whole. While it might seem 'easy' to fix a mod, the development team is mostly focused on the core game itself. This is why mod maintainers are necessary. +Furthermore, mods which are in mainline but are not working correctly for whatever reason reflect poorly on the project on the whole. While it might seem 'easy' to fix a mod, the development team is mostly focused on the core game itself. This is why mod maintainers are necessary. ## How can mods be rescued from removal? From 12fc49173c4e687443d01dd6db6bf9a00c125189 Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 13:49:07 -0800 Subject: [PATCH 6/8] Update doc/MAINLINE_MODS.md Co-Authored-By: I-am-Erk <45136638+I-am-Erk@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index 53d8a58140df2..7a83d2c735552 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -40,7 +40,7 @@ Furthermore, there are additional criteria: ## When do mods get removed? -Mods that have no maintainer or have proven to be consistently some type of maintenance burden are potentially subject to removal. Additionally, mods which had a purpose which is no longer relevant (development mods whose accompanying feature has been finished) will generally be removed once they're no longer deemed necessary. +Mods that have no maintainer or have proven to be consistently some type of maintenance burden are potentially subject to removal. Additionally, mods which had a purpose which is no longer relevant (development mods whose accompanying feature has been finished) will generally be removed once they're no longer deemed necessary. If a maintainer can't be reached for comment on a bugfix or review on a PR'd change for about a month, and hasn't left any indication of when they'll be back, developers will assume they've left the project, and the mod is orphaned. Whenever a stable release occurs, mods which no longer qualify for inclusion will be marked 'obsolete'. This doesn't remove them from the repository, but it does prevent it from being used in new worlds (barring additional effort on the part of the player). From dfbe8c9df262eac85fd118ac6a1116e14e944a27 Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 13:49:15 -0800 Subject: [PATCH 7/8] Update doc/MAINLINE_MODS.md Co-Authored-By: I-am-Erk <45136638+I-am-Erk@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index 7a83d2c735552..6c6a14cb45110 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -52,6 +52,6 @@ Furthermore, mods which are in mainline but are not working correctly for whatev ## How can mods be rescued from removal? -If the mod otherwise meets inclusion criteria but lacks a maintainer, it's as simple as having someone else step forward as the new maintainer. +If the mod otherwise meets inclusion criteria but lacks a maintainer (ie. has been declared orphaned), it's as simple as having someone else step forward as the new maintainer. Otherwise, it needs to either be made to meet the criteria, or it simply isn't going to be staying in mainline. From 7dee56aa7ea1ac64f6d9cf2b1ac0cc106545c8fa Mon Sep 17 00:00:00 2001 From: eso Date: Tue, 21 Jan 2020 13:52:55 -0800 Subject: [PATCH 8/8] Update doc/MAINLINE_MODS.md Co-Authored-By: I-am-Erk <45136638+I-am-Erk@users.noreply.github.com> --- doc/MAINLINE_MODS.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/doc/MAINLINE_MODS.md b/doc/MAINLINE_MODS.md index 6c6a14cb45110..6c4abec176e21 100644 --- a/doc/MAINLINE_MODS.md +++ b/doc/MAINLINE_MODS.md @@ -36,7 +36,9 @@ Furthermore, there are additional criteria: ## What are the responsibilities of a mod maintainer? -`TBD` +A mod maintainer's role is to curate additions to mods and ensure they fit the concept of the mod. Maintainers are also responsible for at least acknowledging bug reports related to the mod: while maintainers don't have to personally manage every bugfix, if a bug is reported, the maintainer should be around to comment and help to find someone to fix it. + +It's important to understand that if a mod is part of mainline, it is expected that people other than the maintainer are going to contribute to it, as with anything. On github, mod maintainers should Approve or Request Changes on PRs to the mod. If the maintainer has approved it, developers with merge permissions will treat that as a request to merge the changes. ## When do mods get removed?