jekyll/jekyll-seo-tag

Consistent order of keys for JSON-LD

Closed this issue · 9 comments

It seems almost every time I jekyll build, the JSON-LD output appears to have changed when it hasn't.

I use a git-based deployment and it'd be nice to not have every page on the site change for every build to make it easier to review beforehand or afterwards, as well as for size of the repository over time.

I suspect the issue is with the key order not being deterministic. Would you be open to a pull request that makes this deterministic?

cap

@Krinkle What version of Jekyll and the SEO-tag plugin are you using..?

    jekyll (4.0.0)
    jekyll-seo-tag (2.6.1)

Is there a difference when you use Jekyll v3.8.6 instead..?

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

I've only tried with jekyll 4.x and have since added the minimal bits I needed directly into my theme.

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

@Krinkle Ever found a solution for this? Doing Rsync for site deployment always uploads all index.html files due to this.

@bmcage I stopped using the plugin and added the bits I actually wanted directly to my theme, without external dependencies (source).