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

Added OrderBy when iterating over tags and keys #2

Closed

Conversation

mrange
Copy link
Contributor

@mrange mrange commented Nov 24, 2023

I think ordering on keys could help eliminate fake diffs because of random shuffling. Perhaps the original order is important to preserve but I coudn't think of a reason why

@devlead
Copy link
Owner

devlead commented Nov 28, 2023

Thanks for your contribution, I'm going ahead and closing this PR.

I've added the following settings which should add similar behaviour

[CommandOption("--skip-frontmatter")]
public bool SkipFrontmatter { get; set; }
[CommandOption("--skip-frontmatter-summary")]
public bool SkipFrontmatterSummary { get; set; }
[CommandOption("--skip-frontmatter-modifed")]
public bool SkipFrontmatterModified { get; set; }
[CommandOption("--skip-frontmatter-order")]
public bool SkipFrontmatterOrder { get; set; }

Also switched tags and properties to an ordered dictionary so they should always be rendered in the same order.

@devlead devlead closed this Nov 28, 2023
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