Difference between revisions of "Meetings"
(Move to past meetings) |
|||
Line 22: | Line 22: | ||
'''PR discussion/reviews''' | '''PR discussion/reviews''' | ||
− | * . | + | * [https://github.com/minetest/minetest/pull/12728 Consolidate API object code] - concept approval requested by TurkeyMcMac (Zughy) |
Also consider: | Also consider: |
Revision as of 21:24, 26 September 2022
Core developer meetings are held when needed. They take place in the #minetest-dev IRC channel.
This Wiki page contains the meeting plans for upcoming meetings, and meeting notes for previous meetings.
Only core devs should edit this page. You can raise points to be discussed in an upcoming meeting by contacting a core dev, or messaging on IRC. Please avoid derailing meetings by posting unrelated things whilst discussion about other things is happening.
Upcoming Meetings
Remember to post meetings in the GitHub org in advance
2022-10-09
Add your points here. Most important comes first.
Organization Discussion
- ...
PR discussion/reviews
- Consolidate API object code - concept approval requested by TurkeyMcMac (Zughy)
Also consider:
- "One Approval" PRs and decide on whether to merge, request changes or close.
- "Roadmap: needs approval" PRs
Past Meetings
2022-09-25
PR discussion/reviews
- Set visibility of players/hide them from others has been rebased twice, it's from 2020. Can anyone have a look at it? Or put it in the 5.7.0 milestone, I don't know (Zughy)
- New physics modifiers concept discussion (Wuzzy)
- The same goes for Show relative screenshot filename on chat when applicable (Zughy)
2022-09-11
- Item pickup general concept discussion
- Item movement unit tests required for bugfix PR #12595
2022-08-28
Organization Discussion
- Add some bugs to the 5.6.1 milestone? (Proposal: each core dev pick one and adds it)
- Two years have passed since the first roadmap brainstorm: create another one to then follow after 5.7? Also to gather feedback
PR discussion/reviews
- Use .md extension for markdown files - yes or no?
- Doc of set_sun/-moon: Absolute orientation and difference in scale - document difference in size or slightly change the scale of both the celestial bodies?
2022-08-14
PR discussion/reviews
- https://github.com/minetest/minetest/issues?q=is%3Aopen+is%3Aissue+label%3AQuestion
- Add minetest.get_player_window_information() - any requirements for merge?
- Settings redesign - how to resolve issues with focus?
2022-07-31
Organization Discussion
- Who writes the release changelog?
- When does the release happen? presumably today
PR discussion/reviews
- Work on merging the last PRs left in the milestone
2022-07-17
PR discussion/reviews
- review 5.6.0 Milestone
Also consider:
- "One Approval" PRs and decide on whether to merge, request changes or close.
- in particular decide whether we want them for 5.6
2022-07-03
Organization Discussion
- Feature freeze date for 5.6.0
- Starting on June 17. Close to 6 months release cycle.
PR discussion/reviews
- Add minetest.get_player_window_information()
- How to resist fingerprinting? -> minor concern, rounding is possible
- Move to player ObjectRef? -> player-specific function. not necessarily.
- Should this be prioritised for 5.6? ->
- Show dep errors in Select Mods modal - pls someone review
- review shall be done
- Settings redesign - concept approval, prioritisation, requirements for submission
- "yes but not in 5.6"
- Bouncy improvements - makes trampolines fun again.
- review shall be done
- repeat_place_time - what do to about this?
- Krock: write a bounds check or removal PR
Roadmapped
- FOV mouse sensitivity
- http client fetch
- not needed
- fix acceleration / fix gravity calculation
- approved
- allow server to use media from texture_path
- flawed, but possibly helpful
- Only count entity as "on ground" if they are colliding on the Y axis
- Krock + sfan5: rejected.
- userdata command line option
- OK, but needs changes
- Add keep_newlines argument to wrap_text
- OK, might need fixing
- Add an item pick up callback (2)
- seems useful.
2022-06-05
Organization Discussion
- make a 5.5.2 release due to security fixes?
- yes
PR discussion/reviews
- Make minetest.write_json use empty list instead of "null" for empty tables - is this approach good?
- Raise default windowed resolution to 1280x720 (720p) - yes or no?
- no
2022-05-22
PR discussion/reviews
- #11545 <- someone should review this --Sfan5 (talk)
- x2048
- #12367 some questions here
- mostly answered before meeting
- https://github.com/minetest/minetest/issues/12353 -- Fix acceleration by appgurueu
2022-05-08
Organization Discussion
- Gauge interest in 5.5.1 release
- 3 in favor, 1 neutral
- Prioritize things for 5.6?
- planned date: July, PRs will be added to milestone
PR discussion/reviews
- #11251 (comment) - backwards compat hinders refactors, drop compat for old irrmt revisions this time?
- <+rubenwardy> anyway. I think the conclusion is that breaking it is fine, with some concerns over ease-of-use during dev
Also consider:
- "One Approval" PRs and decide on whether to merge, request changes or close.
- "Roadmap: needs approval" PRs
- oldest PRs
2022-04-24
Organization Discussion
- What to do with closing out old PRs? Should we start assessing old PRs under the roadmap / assigning people?
- Yes, start marking PRs as supported by core dev / roadmap / needs approval. https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+-label%3ARoadmap+-label%3ABugfix+-label%3A%22Supported+by+core+dev%22+sort%3Acreated-asc+
- Release in June?
- Potentially, will discuss 5.6.0 goals in next meeting
Issues/planning
- See proposal in Optional dependencies being overridden/turned into hard dependencies
- Accepted
- Calling set_detach() + set_properties() + set_pos() on the same step fails to teleport the player half of the time, especially online
- Marked as confirmed, to be investigated
PR discussion/reviews
- Close Shader improvements: Saturation Optimization?
- Closed.
Roadmap: needs approval
- Single functions to get/set every celestial vault element
- Approved.
- Make debug.g/setmetatable respect the "__metatable_debug" metatable field
- Postponed for further inputs.
- WIP: Do not stat /etc/localtime all the time
- Needs actions or will be closed.
- Add mods list formspec display for /mods cmd
- Approved but needs a better implementation.
High-level/Concept approval for:
- Add support for translating content title and descriptions
- Inputs provided.
- Add login/register dialog to separate login/register
- Inputs provided.
Other:
- New spatial index library for entity lookup performance improvements 12040
- Library needs checking, first PR should also replace existing spatial lib
- Add bulk ABMs by TurkeyMcMac
- Unsure of the question, skipped
2022-04-10
Organization Discussion
- More regular meetings
- agreed on fortnightly meetings, starting 2022-04-10 and 2022-04-24
- Review roadmap rules. Remove 1 week limit, use meetings instead
- raise limit to 1 month, attempt review sooner during regular meetings
PR discussion/reviews
Also have a look at "One Approval" PRs and decide on whether to merge, request changes or close.
- SQLite media cache
- Closed for lack of benchmarks. Opinion is split on whether sqlite is a good approach
- proller's breaking world
- Closed due to scale of changes vs prioritisation. Agreement made that a hard network break should not be done
- Add API function minetest.activate_objects_in_area
- worth more consideration, posted comment in PR
- CSM settings
- Supported by core dev
- Add text tile modifier
- Closed as draft and bad approach
- Restore and enhance bouncy behavior by pecksin
- Supported by core dev
- Dual Wielding
- concept accepted, no need for an arbitrary amount of wields
- A light_source should not override sunlight
- Krock brought up an implementation concern
- Add rotation support for wallmounted nodes in 'ceiling' or 'floor' mode
- concept accepted
MTG discussion/reviews
- Don't block fluid (water, lava) flow by flowers, mushrooms, grass and saplings
- Does this break builds? Does this count as a feature? -> Closed as too close to a feature
2021-09-04
Add your points here. Most important comes first.
Organization Discussion
PR discussion/reviews
- Android update for November
- WIP. Waiting for rubenwardy. Not very urgent yet.
- Irrlicht milestone
- Await hecks' OpenGL rewrite until beginning of October.
- .. otherwise remove shadowmap main menu setting and disable within C++
- ItemStack metadata network optimizations - https://github.com/minetest/minetest/pull/11014
- Closed.
- Automatic selection of mods - Adopt or Won't add - https://github.com/minetest/minetest/pull/11274
- Closed.
- Death item duplication and callback fix - https://github.com/minetest/minetest/pull/11445
- Joystick controls - https://github.com/minetest/minetest/pull/11262
- New movement tweak (trivial) - https://github.com/minetest/minetest/pull/11367
- To be discussed later.
- Perhaps provide an alternative fix for https://github.com/minetest/minetest/issues/9725
- TODO reviews
- Dynamic Media V2 https://github.com/minetest/minetest/issues/11550
- Animated particle spanwers https://github.com/minetest/minetest/issues/11545
Also have a look at "One Approval" PRs and decide on whether to merge, request changes or close.
MTG discussion/reviews
2021-01-23
Organization Discussion
- SDL2 integration into Irrlicht
- Ensure that adding an "original Irrlicht" compatibility mode is somehow possible if necessary (ex. Android). (mentioned by sfan5)
- Development on a separate branch to merge when it's usable
- SDL2 as Irrlicht device. WIP (nerzhul) https://github.com/minetest/minetest/pull/10780
- SDL2 for text input. WIP (numzero)
- Plans for 5.4.0 and feature freeze
- Feature freeze on 30 Jan 2021
PR discussion/reviews
- https://github.com/minetest/minetest/issues/9352 get_player_information issue
- No solution yet.
- https://github.com/minetest/minetest/pull/10693 Builtin translate
- Needs re-review. Soon ready for merge prior 5.4.0 freeze
- https://github.com/minetest/minetest/pull/10636 Android, ready for review.
- Gradle error is fixed in master (rebase)
- rubenwardy will review/test
- https://github.com/minetest/minetest/pull/10083 Inventory slots style
- Waiting for merge or another approval (v-rob)
- https://github.com/minetest/minetest/issues/10555 Game settings bug
- Fix Settings for once and all in 5.4.0: https://github.com/minetest/minetest/pull/10819
- https://github.com/minetest/minetest/pull/10265 Formspec image features
- Considered for 5.4.0 prior freeze, but not priority
MTG discussion/reviews
- https://github.com/minetest/minetest_game/pull/2803 Texture compression issue
- Based on issue https://github.com/minetest/minetest_game/issues/2801
- imagefilters.cpp imageCleanTransparent() might need fixing to get white-filled opaque leaves after adding colors to the texture
- Affected textures are whitelisted, but ExeVirus review that again
2020-12-19
Organization Discussion
- 5.3.1 or 5.4.0 release date? There have been multiple big security fixes.
- Perhaps January, depending on how quickly the milestone issues can be resolved
- Texture alpha warnings issue: sfan5 needs time to work on it
- game settings fallback issue: needs decision whether to fix overrideDefaults() or the whole Settings structure (https://github.com/minetest/minetest/issues/10555)
PR discussion/reviews
Various concepts to judge:
- https://github.com/minetest/minetest/pull/10384 image transparency
- https://github.com/minetest/minetest/pull/10430 mapgen constant
- https://github.com/minetest/minetest/pull/10516 base64 (2)
MTG discussion/reviews
2020-10-03
Logs: http://irc.minetest.net/minetest-dev/2020-10-03#i_5738345
Organization Discussion
- 5.4.0 release plan -> December? https://github.com/minetest/minetest/milestone/17
- -> Rough approximated date for release (after 6 months)
- Roadmap - everyone list 3 goals for development (ie: Improve UI), this can then be condensed into a roadmap of goals.
- Add trusted contributors as issue triagers. https://github.com/orgs/minetest/teams/engine/discussions/25
- Suggested users: Wuzzy, Calinou. Wuzzy was already offered core dev, but refused.
- They would help ensure issue quality (asking for information, editing, closing duplicated) and oragnise issues (labelling, prioritising)
- This doesn't prevent core developers from doing the above.
- celeron55 is willing to trial this, paramat is against this.
- -> Krock, rubenwardy: Demo run with a few chosen people. Rules will be written down during this phase to ensure everything's going well. Needs discussion with paramat.
PR discussion/reviews
- Decide on a "use_texture_alpha" solution: https://github.com/minetest/minetest/issues/10342
- -> sfan5 will take care of it
Adopt or modify anticheat fix: https://github.com/minetest/minetest/pull/10340- Require debug priv to view gameplay-relevant debug info (2nd try), add wireframe priv https://github.com/minetest/minetest/pull/9315
- Requested by GreenXenith
- -> Will be reviewed...
MTG discussion/reviews
2020-08-01
Organisation Discussion
- Theme of 5.4.0 (ie: what to focus on)
- New core developers
- New member invited
- Minimal/Devtest policy: https://github.com/minetest/minetest/issues/9645
- Wrote comment
- minetest-mods.github.io needs a library update - could there be any problems? volunteers?
- postpone, not so relevant
PR discussion/reviews
- HUD minimap: https://github.com/minetest/minetest/pull/9079
- HUD compass: https://github.com/minetest/minetest/pull/9312
- Relative ~coordinates: https://github.com/minetest/minetest/pull/9588
- LuaEntitySAO API: https://github.com/minetest/minetest/pull/9717
- Massive PR, but stalls. Needs clarification what to do
- Incompatible with the recent on_step moveresult changes
- Krock: Some API functions might be superfluous (not generic enough), check later again
MTG discussion/reviews
- Wall extension (trivial): https://github.com/minetest/minetest_game/pull/2237
- Craftguide: https://github.com/minetest/minetest_game/pull/2396
2020-06-06
Organisation Discussion
- Allow or deny UTF-8 characters in the source code? relevant PR
- Many different encodings. Non-ASCII characters should be avoided http://irc.minetest.net/minetest-dev/2020-06-06#i_5698515
- Translations and minetest.conf.example need updating. Volunteers?
- Reminder. Will be done when feature freeze starts.
- Feature freeze date
- Bone rotation issues need to be fixed (or reverted)
- Freeze in 1 week: 13 Jun (2 weeks for bugfixes after)
PR discussion/reviews
- POC: Introduce SerializeStream helper class
- Split input/output stream and polish for a real PR.
Suspected issues
- network desync
- sfan5: http://irc.minetest.net/minetest-dev/2020-06-06#i_5698555 not important for release, networking limits
- Wireframe glitch (graphics related)
- Driver issue. Wait for AMD to fix their stuff
2020-03-07
Most important first.
Organisation Discussion
- Release date for 5.2.0: March 14, if possible
- Open issues: https://github.com/minetest/minetest/issues?q=is%3Aissue+is%3Aopen+label%3ABlocker
- Sky fix: https://github.com/minetest/minetest/pull/9472
- Entity shader fix: (none yet)
PR discussion/reviews
- Delayed for 5.3.0: Android XXL PR https://github.com/minetest/minetest/pull/9066
- Fix dependencies in https://github.com/minetest/minetest_android_deps_binaries
- Startup crash needs fixing
- Build script works (sfan5)
MTG discussion/reviews
- Probably 5.3.0 or later: Craft guide: https://github.com/minetest/minetest_game/pull/2396
- Should this be added in 5.2 or not?
2020-01-11
Arranged by: Krock
Most important first.
Organisation Discussion
- Looking for new Core Developer members
- What's missing for 5.1.1?
PR discussion/reviews
- Formspec text cursor feature https://github.com/minetest/minetest/pull/8665
- Scroll container https://github.com/minetest/minetest/pull/9101
- Images in tabs https://github.com/minetest/minetest/pull/8621
MTG discussion/reviews
- Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396
2019-08-10
Arranged by: Krock
PR discussion/reviews
- Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688
- Should this be closed?
- Fix debug.txt growing bigger and bigger https://github.com/minetest/minetest/pull/8382
- What to change? Or reject & close?
- Punchwear https://github.com/minetest/minetest/pull/6804
- Who wants to adopt it? The feature was requested many times.
- Add support for set_formspec_prepend in main menu https://github.com/minetest/minetest/pull/8611
- Hide chat when console is open https://github.com/minetest/minetest/pull/8656
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
2018-July-August
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)
2018-06-??
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)
Template
Add your points here. Most important comes first.
Organization Discussion
- ...
PR discussion/reviews
- ...
Also consider:
- "One Approval" PRs and decide on whether to merge, request changes or close.
- "Roadmap: needs approval" PRs
- oldest PRs. (Roadmap approval for old PRs?)
MTG discussion/reviews
- ...