Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

ci: temporarily disable macos workflow#3270

Merged
jaideepr97 merged 1 commit intoinstructlab:mainfrom
jaideepr97:disable-macos-tmp
Apr 9, 2025
Merged

ci: temporarily disable macos workflow#3270
jaideepr97 merged 1 commit intoinstructlab:mainfrom
jaideepr97:disable-macos-tmp

Conversation

@jaideepr97
Copy link
Copy Markdown
Contributor

temporarily disable macOS workflow until it can be sufficiently debugged without blocking high priority PRs

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@mergify mergify Bot added CI/CD Affects CI/CD configuration ci-failure PR has at least one CI failure labels Apr 9, 2025
@booxter
Copy link
Copy Markdown
Contributor

booxter commented Apr 9, 2025

We should at least report a bug to re-enable the tests explaining how it fails right now; then link to the bug from this patch.

@mergify mergify Bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 9, 2025
@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 9, 2025
Copy link
Copy Markdown
Contributor

@ktdreyer ktdreyer left a comment

Choose a reason for hiding this comment

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

I'll approve. In the future I recommend deleting config sections entirely so it is easier to read and cherry-pick changes.

@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Apr 9, 2025
@jaideepr97
Copy link
Copy Markdown
Contributor Author

I'll approve. In the future I recommend deleting config sections entirely so it is easier to read and cherry-pick changes.

understood!

@jaideepr97
Copy link
Copy Markdown
Contributor Author

@ktdreyer updated

@mergify mergify Bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 9, 2025
@jaideepr97
Copy link
Copy Markdown
Contributor Author

We should at least report a bug to re-enable the tests explaining how it fails right now; then link to the bug from this patch.

#3273

@jaideepr97 jaideepr97 merged commit 2c49016 into instructlab:main Apr 9, 2025
13 of 18 checks passed
@courtneypacheco
Copy link
Copy Markdown
Contributor

@mergify backport release-v0.23

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 10, 2025

backport release-v0.23

✅ Backports have been created

Details

@courtneypacheco
Copy link
Copy Markdown
Contributor

courtneypacheco commented Apr 10, 2025

@mergify backport release-v0.24

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 10, 2025

backport -release-v0.24

❌ No backport have been created

Details
  • Backport to branch -release-v0.24 failed

GitHub error: Branch not found

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 10, 2025

backport release-v0.24

✅ Backports have been created

Details

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

Labels

CI/CD Affects CI/CD configuration ci-failure PR has at least one CI failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants