Skip to content

feat: ✨ add Layout chapter for structure of package#9

Open
lwjohnst86 wants to merge 21 commits into
mainfrom
feat/add-layout-of-package-chapter
Open

feat: ✨ add Layout chapter for structure of package#9
lwjohnst86 wants to merge 21 commits into
mainfrom
feat/add-layout-of-package-chapter

Conversation

@lwjohnst86

Copy link
Copy Markdown
Member

Description

This lists all the files and folders within a data package, as a high-level overview of a data package but also of the book itself.

The PR is a bit on the long side, but I thought it was ok because it isn't high priority to review and merge in.

Needs a thorough review.

Checklist

  • Ran just run-all

dependabot Bot and others added 17 commits July 10, 2026 16:26
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

@signekb signekb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very nice overview 🌟 When reading, I thought that I might as well give some suggestions:

Comment thread .config/cog.toml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ideally, this and the workflow updates shouldn't have been a part of this PR.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

what happened!!!!!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Not sure what happened there. But all those extra files were from main, which I just updated from. Still not sure why this file is here 🤔

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oh, now I know. cocogitto was giving errors, so this cog.toml change was the old version and isn't needed, so I will remove it.

Comment thread docs/config.qmd

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I guess this should also have been a separate PR

Comment thread .zenodo.json
Comment on lines 9 to 10

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same with these changes. Could have been in a separate PR.

Comment thread docs/layout.qmd
Comment thread docs/layout.qmd Outdated
Comment thread docs/layout.qmd Outdated
Comment thread docs/layout.qmd Outdated
Comment thread docs/layout.qmd Outdated
Comment thread docs/layout.qmd Outdated
Comment thread docs/layout.qmd Outdated
@github-project-automation github-project-automation Bot moved this from In Review to In Progress in Product development Jul 14, 2026

@signekb signekb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very nice overview 🌟 When reading, I thought that I might as well give some suggestions:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants