Skip to content

Remove baseclass names from subclass entries. - #326

Merged
Cphrampus merged 1 commit into
Cphrampus:betafrom
153957:subclasses
May 14, 2019
Merged

Remove baseclass names from subclass entries.#326
Cphrampus merged 1 commit into
Cphrampus:betafrom
153957:subclasses

Conversation

@153957

@153957 153957 commented May 9, 2019

Copy link
Copy Markdown

This prevents such sections as being marked as duplicates.

The baseclass <name> tag in the subclass <class> elements caused subclasses to be marked as duplicates are excluded from the compendium.

This happens when using --subtype-format usable.

This prevents such sections as being marked as duplicates.
Cphrampus
Cphrampus previously approved these changes May 9, 2019
@Cphrampus

Copy link
Copy Markdown
Owner

I had not really played around with that flag, I mostly use excludes or includes.

Great catch!

@153957

153957 commented May 14, 2019

Copy link
Copy Markdown
Author

Oops, I accidentally pushed the wrong thing to this branch, causing your review to be invalidated. Please review again and feel free to merge. @Cphrampus

@Cphrampus
Cphrampus self-requested a review May 14, 2019 23:58
@Cphrampus

Copy link
Copy Markdown
Owner

Thanks for finding and fixing this!

@Cphrampus
Cphrampus merged commit 509c101 into Cphrampus:beta May 14, 2019
@153957
153957 deleted the subclasses branch June 2, 2019 19:34
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