Skip to content

ci(main): fix workflow#78

Merged
michaelwittwer merged 6 commits intomainfrom
#77-fix-workflow
Mar 17, 2026
Merged

ci(main): fix workflow#78
michaelwittwer merged 6 commits intomainfrom
#77-fix-workflow

Conversation

@mumenthalers
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 06:48
Copy link

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 updates the main GitHub Actions workflow to allow skipping CI runs via commit message and to prevent downstream jobs from running when skipped.

Changes:

  • Added a checkExecution job using shiftcode/github-action-skip to decide whether the workflow should run.
  • Gated test and build jobs on checkExecution.outputs.shouldExecute.
  • Adjusted build job permissions and introduced GH_TOKEN env for the publish step.

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

Copilot AI review requested due to automatic review settings March 17, 2026 07:12
 - @shiftcode/airtable-mapper@1.0.0-pr77.0
 - @shiftcode/airtable-syncer@1.0.0-pr77.0
 - @shiftcode/branch-utilities@6.0.0-pr77.0
 - @shiftcode/cdk-utilities@1.0.0-pr77.0
 - @shiftcode/eslint-config-recommended@7.0.0-pr77.0
 - @shiftcode/eslint-plugin-rules@6.0.0-pr77.0
 - @shiftcode/iac-utilities@1.0.0-pr77.0
 - @shiftcode/inversify-logger@1.0.0-pr77.0
 - @shiftcode/lambda-utilities@1.0.0-pr77.0
 - @shiftcode/logger@5.0.0-pr77.0
 - @shiftcode/nest-logger@1.0.0-pr77.0
 - @shiftcode/publish-helper@6.0.0-pr77.0
 - @shiftcode/utilities@5.0.0-pr77.0
Copy link

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 updates the main CI workflow to conditionally skip execution based on the commit message, and wires that decision into both the test and build/publish jobs.

Changes:

  • Add a checkExecution job to decide whether the workflow should run (based on [skip_build]).
  • Gate test and build jobs on checkExecution’s shouldExecute output.
  • Provide GH_TOKEN to the publish step to support protected-branch commits and follow-up workflow runs.

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

@michaelwittwer michaelwittwer self-requested a review March 17, 2026 16:08
mumenthalers and others added 2 commits March 17, 2026 17:25
 - @shiftcode/airtable-mapper@1.0.0-pr77.1
 - @shiftcode/airtable-syncer@1.0.0-pr77.1
 - @shiftcode/branch-utilities@6.0.0-pr77.1
 - @shiftcode/cdk-utilities@1.0.0-pr77.1
 - @shiftcode/eslint-config-recommended@7.0.0-pr77.1
 - @shiftcode/eslint-plugin-rules@6.0.0-pr77.1
 - @shiftcode/iac-utilities@1.0.0-pr77.1
 - @shiftcode/inversify-logger@1.0.0-pr77.1
 - @shiftcode/lambda-utilities@1.0.0-pr77.1
 - @shiftcode/logger@5.0.0-pr77.1
 - @shiftcode/nest-logger@1.0.0-pr77.1
 - @shiftcode/publish-helper@6.0.0-pr77.1
 - @shiftcode/utilities@5.0.0-pr77.1
Copilot AI review requested due to automatic review settings March 17, 2026 16:29
@mumenthalers mumenthalers review requested due to automatic review settings March 17, 2026 16:29
Copilot AI review requested due to automatic review settings March 17, 2026 16:31
 - @shiftcode/airtable-mapper@1.0.0-pr77.2
 - @shiftcode/airtable-syncer@1.0.0-pr77.2
 - @shiftcode/branch-utilities@6.0.0-pr77.2
 - @shiftcode/cdk-utilities@1.0.0-pr77.2
 - @shiftcode/eslint-config-recommended@7.0.0-pr77.2
 - @shiftcode/eslint-plugin-rules@6.0.0-pr77.2
 - @shiftcode/iac-utilities@1.0.0-pr77.2
 - @shiftcode/inversify-logger@1.0.0-pr77.2
 - @shiftcode/lambda-utilities@1.0.0-pr77.2
 - @shiftcode/logger@5.0.0-pr77.2
 - @shiftcode/nest-logger@1.0.0-pr77.2
 - @shiftcode/publish-helper@6.0.0-pr77.2
 - @shiftcode/utilities@5.0.0-pr77.2
Copy link

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

Updates the main CI workflow to support skipping execution via commit message and to improve publishing behavior, while also rebasing multiple workspace packages onto a new pr77.1 prerelease version set.

Changes:

  • Added a checkExecution job to conditionally run test/build based on commit message ([skip_build]).
  • Updated the build job to depend on the new skip-check and to pass GH_TOKEN for publishing/git operations.
  • Bumped multiple package versions and internal workspace dependency ranges to *-pr77.1 (including corresponding package-lock.json updates).

Reviewed changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.github/workflows/main.yml Adds commit-message skip gate and passes GH_TOKEN into publish step.
package-lock.json Updates lockfile entries to reflect new prerelease versions/ranges.
packages/airtable-mapper/package.json Bumps package version to pr77.1.
packages/airtable-syncer/package.json Bumps package version and internal devDependency prerelease ranges.
packages/branch-utilities/package.json Bumps package version to pr77.1.
packages/cdk-utilities/package.json Bumps package version to pr77.1.
packages/eslint-config-recommended/package.json Bumps package version and plugin dependency prerelease range.
packages/eslint-plugin-rules/package.json Bumps package version to pr77.1.
packages/iac-utilities/package.json Bumps package version and internal devDependency prerelease ranges.
packages/inversify-logger/package.json Bumps package version and internal devDependency prerelease range.
packages/lambda-utilities/package.json Bumps package version and internal devDependency prerelease ranges.
packages/logger/package.json Bumps package version/devDependency prerelease range.
packages/nest-logger/package.json Bumps package version and internal devDependency prerelease range.
packages/publish-helper/package.json Bumps package version and internal devDependency prerelease range.
packages/utilities/package.json Bumps package version to pr77.1.

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

You can also share your feedback on Copilot code review. Take the survey.

@michaelwittwer michaelwittwer merged commit d2e4049 into main Mar 17, 2026
@michaelwittwer michaelwittwer deleted the #77-fix-workflow branch March 17, 2026 16:43
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.

4 participants