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

Change head metadata from position: to sidebar_position #19

Merged
merged 2 commits into from
Dec 31, 2023

Conversation

thejcannon
Copy link
Member

Fixes #11. Not sure where I got position from.

Verified with local test.

Copy link
Contributor

@huonw huonw left a comment

Choose a reason for hiding this comment

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

Thanks!

For my education, are the scraping scripts stored somewhere?

@thejcannon
Copy link
Member Author

Not anywhere yet. Was planning on enshrining them in gists.

@thejcannon thejcannon merged commit 6185bd4 into main Dec 31, 2023
1 check passed
@thejcannon thejcannon deleted the jcannon/sidebar-position branch December 31, 2023 02:00
@huonw
Copy link
Contributor

huonw commented Dec 31, 2023

Ah cool. One "silly" method I do for preserving transient stuff like that is open a PR and then immediately close it, then if they're needed in future the PR can be manipulated as appropriate.

That said, it wouldn't be the worst to just commit them to this repo for now, and once they're no longer needed, delete them.

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.

Docs sidebar items are sorted alphabetically inappropriately
2 participants