Skip to content

Automatic set location-set to include planet#2437

Merged
matkoniecz merged 4 commits into
mainfrom
automatic-locationSet-include-planet
Jul 14, 2026
Merged

Automatic set location-set to include planet#2437
matkoniecz merged 4 commits into
mainfrom
automatic-locationSet-include-planet

Conversation

@tyrasd

@tyrasd tyrasd commented Jul 9, 2026

Copy link
Copy Markdown
Member

Replace #2381 with custom logic to automatically add implicit include: ["Planet"] to the dist output if not specified otherwise.

@tyrasd tyrasd added the build Issue concerning build scripts and their "dist" output label Jul 9, 2026
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

🍱 Your pull request preview is ready

Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone.

Use the Compare View on the Tagging Schema Browser (alpha) to see the changes of your PR against the main branch.

FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes.

@matkoniecz

Copy link
Copy Markdown
Collaborator

@tyrasd have you tested it?

@matkoniecz

Copy link
Copy Markdown
Collaborator

@matkoniecz

Copy link
Copy Markdown
Collaborator

I am tempted to merge now and make followup PR for those two fields.

@matkoniecz
matkoniecz merged commit 36785b7 into main Jul 14, 2026
8 checks passed
@matkoniecz
matkoniecz deleted the automatic-locationSet-include-planet branch July 14, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issue concerning build scripts and their "dist" output

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants