Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add inline node inclusion support to mod.yaml. #18591

Merged
merged 1 commit into from Oct 10, 2020

Conversation

pchote
Copy link
Member

@pchote pchote commented Sep 6, 2020

This PR implements my take on #18277. This allows multiple Include: nodes to be defined, inserting the contents of the referenced file at the node's location.

This allows for an IMO more natural syntax:

Metadata:
	...etc...

Include: rules/manifest.yaml
Include: sequences/manifest.yaml
Include: tilesets/manifest.yaml

as well as include/overrides:

Include: ../common/fonts.yaml
Fonts:
	Title:
		Font: ra|ZoodRangmah.ttf
		Size: 48
		Ascender: 26

(although IMO this would be bad practice because ../common/* won't work for SDK based mods with multiple mod roots)

@reaperrr reaperrr merged commit 1ddbe50 into OpenRA:bleed Oct 10, 2020
@pchote pchote deleted the mod-yaml-includes branch December 6, 2020 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants