Skip to content

Separate steps for linting/building#7

Open
sburton84 wants to merge 3 commits intomainfrom
separate-steps
Open

Separate steps for linting/building#7
sburton84 wants to merge 3 commits intomainfrom
separate-steps

Conversation

@sburton84
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR restructures the GitHub Actions release pipelines to separate linting from building (and releasing) for the documentation UI, improving job clarity and allowing lint to gate subsequent release steps.

Changes:

  • Split the Node workflow in release.yml into lint_node and build_and_release_node, with an explicit dependency (needs) between them.
  • Update the PyPI release job dependency to run after the new Node build+release job.
  • Split the Node workflow in pre-release.yml into separate lint and build jobs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/release.yml Splits linting into its own job and gates build+release and subsequent PyPI publishing via needs.
.github/workflows/pre-release.yml Splits lint and build into separate jobs, adding env vars to lint and introducing a new build job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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