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

chore: Bump SDG version to ensure the latest docling changes are consumed - #3281

Merged
courtneypacheco merged 1 commit into
mainfrom
bump-sdg-min-version
Apr 10, 2025
Merged

chore: Bump SDG version to ensure the latest docling changes are consumed#3281
courtneypacheco merged 1 commit into
mainfrom
bump-sdg-min-version

Conversation

@courtneypacheco

@courtneypacheco courtneypacheco commented Apr 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

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.

Before we cut a release, we should bump the SDG version in the requirements.txt file to ensure that SDG's latest release, which contains important docling bugfixes, gets consumed.

@courtneypacheco
Bump SDG version to ensure the latest docling changes are consumed
8d791aa 
Bump the SDG version in the `requirements.txt` file to ensure that SDG's latest release, which contains important docling bugfixes, gets consumed.

Signed-off-by: Courtney Pacheco <6019922+courtneypacheco@users.noreply.github.com>
@mergify mergify Bot added the dependencies Relates to dependencies label Apr 10, 2025
@ktdreyer

ktdreyer commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

We intend to backport this to release-v0.23 and abandon release-v0.24, right?

@courtneypacheco
courtneypacheco marked this pull request as ready for review April 10, 2025 16:00
@courtneypacheco

courtneypacheco commented Apr 10, 2025

Copy link
Copy Markdown
Contributor Author

@ktdreyer I want to ensure that instructlab-sdg==0.7.3 still runs on main after the recent updates to our CI. instructlab-sdg==0.7.3 was last tested before the CI broke earlier this week. Now that it's fixed again, I want to validate it once more.

@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 10, 2025
@courtneypacheco

courtneypacheco commented Apr 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Merging this to create a new release. We will need to update the approval permissions in the future to only require 1 in these situations.

@courtneypacheco
courtneypacheco merged commit 5adac3c into main Apr 10, 2025
@courtneypacheco
courtneypacheco deleted the bump-sdg-min-version branch April 10, 2025 16:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@danmcp danmcp danmcp approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

dependencies Relates to dependencies one-approval PR has one approval from a maintainer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@courtneypacheco @ktdreyer @danmcp