Skip to content

feat: beet.contrib.bake_overlays - #468

Open
rx-dev wants to merge 2 commits into
mcbeet:mainfrom
Smithed-MC:feat/contrib-bake-overlays
Open

feat: beet.contrib.bake_overlays#468
rx-dev wants to merge 2 commits into
mcbeet:mainfrom
Smithed-MC:feat/contrib-bake-overlays

Conversation

@rx-dev

@rx-dev rx-dev commented Apr 20, 2025

Copy link
Copy Markdown
Member

Adds beet.contrib.bake_overlays as a new beet plugin to help collapse overlays onto a singular version.

open questions

  • this only handles namespace and ignores extras and namespace_extras
  • the naming of the config is a bit odd
  • if config isn't available, it pulls from the set pack.pack_format which usually pulls from the latest version (actually, if set in beet.json, if a pack.mcmeta exists, it'll still be the latest if it's still unset).
  • the merging temporarily replaces a merge policy with a defunct one. unsure if this is the best route for all merging involved.
  • Make plugin also work for generic packs and not just ctx

@rx-dev

rx-dev commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

@edayot I would appreciate a review for this one!

@edayot edayot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems to be written for beet before the monorepo, might need a rebase or a merge from main

"""

for name, overlay in pack.overlays.items():
if overlay.supported_formats is None:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This does seems to account for min_format / max_format

"""

if opts.selected_data_format is None:
opts.selected_data_format = ctx.data.pack_format

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same here, pack_format can be None now

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.

2 participants