Skip to content

Document and align release packaging behavior#58

Open
fingolfin wants to merge 1 commit intomainfrom
codex/issue-40-readme
Open

Document and align release packaging behavior#58
fingolfin wants to merge 1 commit intomainfrom
codex/issue-40-readme

Conversation

@fingolfin
Copy link
Member

Expand the README section on what release-pkg actually does,
including PackageInfo.g fields, validation checks, archive
creation, GitHub release uploads, dry-run artifacts, and the
separation from update-gh-pages.

Co-authored-by: Codex codex@openai.com

Resolves #40

Expand the README section on what release-pkg actually does,
including PackageInfo.g fields, validation checks, archive
creation, GitHub release uploads, dry-run artifacts, and the
separation from update-gh-pages.

Co-authored-by: Codex <codex@openai.com>
Comment on lines +223 to +225
If `dry-run: false` (the default), the action creates or updates a GitHub
release in the same repository, using the extracted tag, and uploads all
generated archives plus the manual PDFs as release assets.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should perhaps also mention package-info.json?

Comment on lines +227 to +229
If `dry-run: true`, the action does not publish a GitHub release. Instead, it
uploads the generated archives, manual PDFs, and the supplied `body-text` as a
workflow artifact.
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, perhaps mention package-info.json?

Comment on lines +231 to +232
This action does not build package manuals; do that in a separate workflow step,
for example via [build-pkg-docs][3].
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe worth specifying it has to happen in a prior workflow step rather than just separate, since this action assumes the documentation is already present.

The copy step uses `cp -r . ...`, so top-level dotfiles and dot-directories are
copied into the temporary release tree as well.

After copying, the action removes some release-irrelevant files from the copied
Copy link
Member

Choose a reason for hiding this comment

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

Maybe worth linking to code where it happens?
This catalog is not closed, I think.

This action does not build package manuals; do that in a separate workflow step,
for example via [build-pkg-docs][3].

This action also does not update a package website or `gh-pages` branch. That
Copy link
Member

Choose a reason for hiding this comment

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

I would write a sentence or two that this actions is prerequisite to update-gh-pages.

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.

Document what this action actually does

3 participants