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

Updating chaos basic selections for new 9e codex #54

Merged
merged 2 commits into from
Jul 26, 2022

Conversation

JLoach
Copy link
Contributor

@JLoach JLoach commented Jul 23, 2022

Updating a few 9e codex name differences in the basic selections, there are probably a lot more, these are just the ones I noticed so far, I will update others as I see them.

@maybe-hello-world
Copy link
Owner

I suggest adding Excruciator cannons from Venomcrawler too

@JLoach
Copy link
Contributor Author

JLoach commented Jul 23, 2022

For some reason, excruciator cannons already do not get added to the output when I test it, so not needed?

@maybe-hello-world
Copy link
Owner

Oh, found it.
Currently, there're chaos.yml and csm.yml files. csm.yml is for units that are in CSM codex only and chaos for units that can be in different codices (1000sons, deathguard, daemons, etc.). As venomcrawler is a csm-only unit, it's in csm.yml.

May I ask you to move master of possession and venomcrawler info to csm.yml?
Thanks.

Copy link
Owner

@maybe-hello-world maybe-hello-world left a comment

Choose a reason for hiding this comment

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

See the comment above.

@JLoach
Copy link
Contributor Author

JLoach commented Jul 26, 2022

Moved those to the correct file, sorry about that.

@maybe-hello-world
Copy link
Owner

No worries, thank you so much for the help :)

@maybe-hello-world maybe-hello-world merged commit 934603f into maybe-hello-world:master Jul 26, 2022
This pull request was closed.
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