Difference between revisions of "Meetings"
(Update after meeting) |
Rubenwardy (talk | contribs) |
||
Line 1: | Line 1: | ||
= Upcoming Meetings = | = Upcoming Meetings = | ||
− | == 2019-06- | + | == 2019-07-X == |
+ | |||
+ | '''PR discussion/reviews''' | ||
+ | |||
+ | * Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688 | ||
+ | ** Should this be closed? | ||
+ | * Punchwear https://github.com/minetest/minetest/pull/6804 | ||
+ | ** Who wants to adopt it? The feature was requested many times. | ||
+ | * InvRef: Add add_list https://github.com/minetest/minetest/pull/8466 | ||
+ | ** Wanted feature or useless overhead? | ||
+ | * Custom item wear display https://github.com/minetest/minetest/pull/8447 | ||
+ | ** Concept discussion. Acceptable PR or is there any better solution? | ||
+ | * /help formspec https://github.com/minetest/minetest/pull/8385 | ||
+ | ** Concept discussion. Acceptable PR or what to change? | ||
+ | |||
+ | '''MTG discussion/reviews''' | ||
+ | * Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396 | ||
+ | * Craftguide from LoneWolfHT https://github.com/minetest/minetest_game/pull/2394 | ||
+ | |||
+ | |||
+ | = Past Meetings = | ||
+ | == 2019-06-29 == | ||
+ | |||
Arranged by: rubenwardy | Arranged by: rubenwardy | ||
+ | |||
+ | Logs: http://irc.minetest.net/minetest-dev/2019-06-29#i_5561978 | ||
'''Organisation discussion''' | '''Organisation discussion''' | ||
Line 26: | Line 50: | ||
* list spacing https://github.com/minetest/minetest/pull/7971 (perhaps integrate into style[]?) | * list spacing https://github.com/minetest/minetest/pull/7971 (perhaps integrate into style[]?) | ||
* Unarelith's refactors | * Unarelith's refactors | ||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
− | |||
== July-August 2018 == | == July-August 2018 == |
Revision as of 20:01, 7 July 2019
Upcoming Meetings
2019-07-X
PR discussion/reviews
- Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688
- Should this be closed?
- Punchwear https://github.com/minetest/minetest/pull/6804
- Who wants to adopt it? The feature was requested many times.
- InvRef: Add add_list https://github.com/minetest/minetest/pull/8466
- Wanted feature or useless overhead?
- Custom item wear display https://github.com/minetest/minetest/pull/8447
- Concept discussion. Acceptable PR or is there any better solution?
- /help formspec https://github.com/minetest/minetest/pull/8385
- Concept discussion. Acceptable PR or what to change?
MTG discussion/reviews
- Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396
- Craftguide from LoneWolfHT https://github.com/minetest/minetest_game/pull/2394
Past Meetings
2019-06-29
Arranged by: rubenwardy
Logs: http://irc.minetest.net/minetest-dev/2019-06-29#i_5561978
Organisation discussion
- Is 5.0.2 needed for the newlines in setting name bug? (https://github.com/minetest/minetest/pull/8590)
- Yet no clear decision
- What is needed to have good mobs in Minetest (- Game)?
- I suggest roping in mob devs like tenplus1 and stujones1
- No blood included. Belongs to another mod
- Combine pathfinder issues into a single issue
- Possible close issues/PRs: https://github.com/minetest/minetest/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+label%3A%22Possible+close%22
- Does 1 disapproval block a PR?
- SmallJoker seems to think it does
Proposed order of merging formspec PRs
The idea is to merge the most important PR first (top of list) to not drown in conflicts: https://github.com/minetest/minetest/projects/6
- style[] https://github.com/minetest/minetest/pull/8383
- scroll_container[] https://github.com/minetest/minetest/pull/8591
- list spacing https://github.com/minetest/minetest/pull/7971 (perhaps integrate into style[]?)
- Unarelith's refactors
July-August 2018
Arranged by: Krock
Organisation discussion
- Will there be a 0.4.18 release?
- http://irc.minetest.net/minetest-dev/2018-07-14#i_5358759 - sounds like a definitive "no"?
- Specify a rough 5.0.0 release date (month, year)
- Freeze for multiple weeks for testing & fixing
- Forums: Clarify which rules (amongst license) apply for WIP modpacks/mods/games
PR discussion/reviews
- Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
- Improves readability - does it need `string.format` for a possibly prettier code?
- Protocol cleanups https://github.com/minetest/minetest/pull/7348
- Review needed (compat break)
June 2018
Arranged by: Krock
Organisation discussion
- Will there be a 0.4.18 release?
- Bugfixing the bugfix release
- Proposal: change master version scheme to: 5.0.0
PR discussion/reviews
- Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
- Improves readability - does it need `string.format` for a possibly prettier code?
- Lua parameter reading https://github.com/minetest/minetest/pull/7410
- Good design concept? Comments?
- Protocol cleanups https://github.com/minetest/minetest/pull/7348
- Review needed (compat break)
- CSM keypress - is this advantage acceptable? https://github.com/minetest/minetest/pull/7008
- Needs flavors to be added: http://irc.minetest.net/minetest-dev/2018-06-11#i_5329651
2018-05-19
Arranged by: Krock
Proposed time: 18:00 UTC
Organisation discussion
- 0.4.17 - backport missing fixes and set date for freeze
- ~Android-specific fix for https://pastebin.com/j4mjniaq must be added too (during freeze?)~
- Needs buildbot fixes (404 packages)
- Backport https://github.com/minetest/minetest/commit/4bb41a19d (http://irc.minetest.net/minetest-dev/2018-05-19#i_5310266)
- 0.4.17 - set release date
PR discussion/reviews
- Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
- Improves readability - does it need `string.format` for a possibly prettier code?
- Fix missing ignore textures (https://github.com/minetest/minetest/pull/7326)
- Needs more dev opinions & reviewing
2018-05-05
Arranged by: Krock
It's now almost a month since the last meeting - it time for the next one. Proposed time: 18:00 UTC
Organisation discussion
- When do we have time to release 0.4.17?
- Still unclear freeze and release dates. 1 MTE needs backporting first
- Whether to leave the issue number link in a commit message when merging on Github
- MTE: Keep the numbers if they're inserted automatically (high PR count, helpful for the web interface blame or fast jumps in commits)
- MTG: The numbers may be removed if wanted (small amount of PRs)
- http://irc.minetest.net/minetest-dev/2018-05-05#i_5298887
PR discussion/reviews
- Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
- Improves readability - does it need `string.format` for a possibly prettier code?
- Abort when trying to set a not registered node (https://github.com/minetest/minetest/pull/7011)
- Needs more dev opinions & reviewing
2018-04-07
Arranged by: Krock
Organisation discussion
- Questions related to 0.4.17, 0.4.18 or future development?
- 0.4.17 release should be moved to 21/22 April for backports (http://irc.minetest.net/minetest-dev/2018-04-07#i_5272834)
- Feature/backport freeze after 14 April to have enough time for testing
PR discussion/review
- Android joystick (https://github.com/minetest/minetest/pull/7126)
- Formspecs: Add a <use_color_alpha> parameter to the box[] element (https://github.com/minetest/minetest/pull/7116)
- Causes errors in-game for older clients but doesn't affect playability - merge or not?
- Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
- Improves readability - does it need `string.format` for a possibly prettier code?
MTG discussion/reviews
- Tools: rebalance (https://github.com/minetest/minetest_game/issues/1681)
- Needs either a levelling system and/or special abilities for different materials to add variety
- paramat suggests removing bronze, which is incorrectly a stronger tool than steel (http://irc.minetest.net/minetest-dev/2018-04-07#i_5272959)
- Future mese concept: http://irc.minetest.net/minetest-dev/2018-04-07#i_5273013
2018-03-31
Arranged by: Krock
PR discussion/reviews
- Introduce clang-tidy to the CI (https://github.com/minetest/minetest/pull/6295)
- Android joystick (https://github.com/minetest/minetest/pull/7126)
- Minor change requests, discussion about joystick placement
0.4.17 release schedule
- Definitive release date
- Announced. 15 April 2018, see https://forum.minetest.net/viewtopic.php?f=18&t=19905
- Any new, important bugfixes to backport?
- Added the `core.rotate_node` bugfix, otherwise probably not. It's pretty much stable.
- Volunteer for cherry-picking the ToDo backport commits (https://github.com/minetest/minetest/pull/6746)
- `<sfan5> probably me` (http://irc.minetest.net/minetest-dev/2018-03-31#i_5264276)
2018-03-10
Arranged by: Krock
Organisation discussion
- New Doxygen rule
- Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249578
- Opposed: paramat, sfan5, SmallJoker, Shara
- Comments in the functions directly are more welcome
- Group issues in projects?
- Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249600
- Purpose is unclear and questionable due missing information, needs to be discussed later again
PR discussion/reviews
- Remove texture pack caching (https://github.com/minetest/minetest/pull/6660)
- Don’t create CSM script env (https://github.com/minetest/minetest/pull/6951)
- Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249625
- Needs improvement, questionable solution due request of server-provided CSM (https://github.com/minetest/minetest/issues/5393)
- “Bugfix” for 0.4-backported rotate_node calls (https://github.com/minetest/minetest/pull/6900)
- SmallJoker will test again and possibly merge after success
- Android C++11 build fix (https://github.com/minetest/minetest/pull/6796)
- Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249684
- sfan5 and nerzhul will try to test it soon. It does not break the regular setup, so a merge is to expect soon
- Confirmed working by Wayward_One (http://irc.minetest.net/minetest-dev/2018-03-10#i_5249720)
- Autogenerated settings header (https://github.com/minetest/minetest/pull/6728)
- Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249703 (stub)
- Needs attention
0.4.17 release schedule
- Definitive release date
- absent nerzhul: “release 0.4.17 ASAP” (http://irc.minetest.net/minetest-dev/2018-03-10#i_5249422)
- Are there any new, important bugfixes to backport?
- Volunteer for cherry-picking the 6 TODO (+ ^ from above) commits (https://github.com/minetest/minetest/pull/6746)
2017-05-20
Minetest developer meeting 2017-05-20 Last chance to merge/improve the remaining PR before feature freeze: Old business: * Additional hooks for inventory manipulations callbacks (#4035) * Krock was working on it (#5647), but had an issue with a mysterious crash bug. Progress? * Timed move (#1489) * Merge/close? * nerzhul: close for me * nore: Merge * Krock: Merge * Split block position in SQLite3 database (and use R*Tree) (#1845) * Rebased, performance tests to do soon(tm). * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) * Add list_predict formspec element (#1988) * sofar worked on a rebase a while ago. Close or merge a modern version? * New screenshots for website (web#78) https://github.com/minetest/minetest.github.io/issues/78 * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots. * > Calinou will handle it New business: * First, release blockers: * https://github.com/minetest/minetest/pull/5767 Don't add damage flash while punch texture modifier is active #5767 * > Bugfix, punted to next meeting. * https://github.com/minetest/minetest/issues/5782 [CSM] Disable preview mod before release #5782 * CSM disabled by default anyways since it's experimental * > Fixed by #5554 * https://github.com/minetest/minetest/issues/5728 Block instadig after tool switch #5728 * > Fixed by #5785 * nerzhul: * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read) * maybe close https://github.com/minetest/minetest/pull/5654 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience) * > Merge with minor changes * Create world better experience: https://github.com/minetest/minetest/pull/5589 * Permit to enable/disable mods on CSM https://github.com/minetest/minetest/pull/5554 (very important) * ShadowNina: CSM is experimental, so I don't think any CSM issues should be blockers * nerzhul: here the problem if a user enable client side modding his experience will be affected by this preview mod, dedicated to modders) * > nerzhul will merge * https://github.com/minetest/minetest/pull/5589 (start world after creation) * LMDB backend (https://github.com/minetest/minetest/pull/4206) * nerzhul: Close? We already have 5 backends to maintain and we already cover all the usecases (client usage, performance usage, reliability usage)
2017-05-13
Minetest developer meeting 2017-05-13 Old business: * Additional hooks for inventory manipulations callbacks (#4035) * Krock was working on it (#5647), but had an issue with a mysterious crash bug. * > No progress * Timed move (#1489) * Merge/close? * > nerzhul: close for me * > nore: Merge * > Krock: Merge * Split block position in SQLite3 database (and use R*Tree) (#1845) * Rebased, performance tests to do soon(tm). * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) * Add list_predict formspec element (#1988) * sofar worked on a rebase a while ago. Close or merge a modern version? New business: * New screenshots for website (web#78) https://github.com/minetest/minetest.github.io/issues/78 * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots. * Fix 'alpha' property for liquid nodes (#5494) * > paramat testing. * CSM: Fix undocumented API calls (#5756) nerzhul: please look at recent PR for this meeting. Feature Freeze is in 8 days Here is a list to look at: * https://github.com/minetest/minetest/pull/5746 (cleanup content_mapblock) * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read) * maybe close https://github.com/minetest/minetest/pull/5654 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience) * Create world better experience: https://github.com/minetest/minetest/pull/5589 * Permit to enable/disable mods on CSM https://github.com/minetest/minetest/pull/5554 (very important)
2017-05-06
Minetest developer meeting 2017-05-06 Old business: "Add configurable key bindings" (#1439) Rebased by nore, needs tweaks and merge. Additional hooks for inventory manipulations callbacks (#4035) Krock is working on it, progress? Timed move (#1489) Merge/close? nerzhul: close for me Use a settings object for the main settings (#1949) Rebased, should be merged soon. Split block position in SQLite3 database (and use R*Tree) (#1845) Rebased, performance tests to do soon(tm). nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) Add list_predict formspec element (#1988) sofar worked on a rebase a while ago. Close or merge a modern version? Use object property "stepheight" for player stepheight too. (#2056) Agreed to merge, but nobody yet assigned. Make players respect makes_footstep_sound in the object properties (#2852) Agreed to merge, nobody assigned. New business: Add support for multiple listen addresses (#2604) Big patch, but can't really be split up much more since everything's interdependent. Make the player collisionbox settable (#2738) Clean up numeric.h and split FaceFositionCache from it (#3256) Up-to-date and looks more-or-less ready to merge, just needs formal approval. <nerzhul>: https://github.com/minetest/minetest/pull/5361 Background color on textarea/field/pwdfield) https://github.com/minetest/minetest/pull/5355: vertical bar in text areas Possible close: https://github.com/minetest/minetest/pull/5281 and https://github.com/minetest/minetest/pull/5279 Documentation update: https://github.com/minetest/minetest/pull/4968 close or not close ? For future meeting: If finished: https://github.com/minetest/minetest/pull/5544 (on_item_use CSM)
2017-04-29
Arranged by: paramat
ShadowNinja is unavailable today, paramat will be hosting in his stead.
Old business: (Things mentioned at previous meetings that haven't yet been resolved)
- "Add configurable key bindings" (#1439)
- Rebased by nore, needs tweaks and merge.
- Additional hooks for inventory manipulations callbacks (#4035)
- Krock is working on it, progress?
- Minetest subgame meeting
- paramat held one at 1900Z
- Timed move (#1489)
- Merge/close?
- Use a settings object for the main settings (#1949)
- Rebased, should be merged soon.
- Split block position in SQLite3 database (and use R*Tree) (#1845)
- Rebased, performance tests to do soon(tm).
- nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
- Add list_predict formspec element (#1988)
- sofar worked on a rebase a while ago. Close or merge a modern version?
New business: (mainly oldest or least recently updated PRs that haven't been handled yet)
- Added formspec element 'KeyEventBox' for charbased input by ninnghazad (#1737)
- Closed: better done with client-side mods.
- Use object property "stepheight" for player stepheight too. (#2056)
- Agreed to merge, but nobody assigned.
- Add mapgen settings to create world dialog by srifqi (#2561)
- Merge, but needs changes.
2017-04-22
Minetest developer meeting 2017-04-22 Old business: "getTime refactoring" (#1188) not yet merged (nore) "Delayed shutdown" merged by nerzhul. "Disable Android LevelDB" rebased by nore and merged by nerzhul. "Add configurable key bindings" (#1439) rebased by nore, needs tweaks and merge. Additional hooks for inventory manipulations callbacks (#4035) Needs rebase and cleanup, volunteers? Rename .txt to .md (#5451)? New business: (PRs obtained by sorting by age on GitHub) Schedule Minetest subgame meeting? Coverity: https://scan.coverity.com/projects/12441 Need to triage issues, many legitimate issues to fix. Timed move (#1489) Added formspec element 'KeyEventBox' for charbased input (#1737) Add ability to escape commands with a back-slash (#1768) <ShadowNinja> I may want to tweak this so that it doesn't strip the slash when you type "\foo" (or maybe not? It should at least work for the client-side dot-commands too) Split block position in SQLite3 database (and use R*Tree) (#1845) Use a settings object for the main settings (#1949) <ShadowNinja> I also unified Initialize and InitiallizeAsync, which was necessary to implement this. I think the unification is worth merging even without the settings object change. This conflicts with just about the entire ScriptAPI, so I don't want to rebase it too often :-) Add listpredict to the formspec (#1988) Add non-global mod namespaces (#2039)