Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 12, 2024

Addresses #365


This is an automatic backport of pull request #366 done by Mergify.

@bbrowning
Copy link
Contributor

The code has changed between releases here, so this automatic backport failed. We'll need to manually adjust the code to work with the older version of SDG here.

@bbrowning
Copy link
Contributor

@aakankshaduggal I adjusted this backport of your PR to work with SDG 0.3.x - it's a pretty straightforward change to remove the use_legacy_pretraining_format stuff, but please review to ensure it looks correct to you.

@aakankshaduggal
Copy link
Member

Thanks @bbrowning, this looks good to me!

@mergify mergify bot added one-approval ci-failure CI/CD Affects CI/CD configuration and removed ci-failure labels Nov 12, 2024
@bbrowning
Copy link
Contributor

Hmm, got caught by some other changes preventing CI from passing on this backport - will have to open a separate backport PR, get that one merged, and then maybe we can get this one green.

@bbrowning bbrowning force-pushed the mergify/bp/release-v0.3/pr-366 branch from 4e10073 to d2a179a Compare November 12, 2024 19:18
@mergify mergify bot removed the ci-failure label Nov 12, 2024
@mergify mergify bot added ci-failure and removed one-approval labels Nov 12, 2024
@bbrowning
Copy link
Contributor

@Mergifyio rebase

@mergify
Copy link
Contributor Author

mergify bot commented Nov 12, 2024

rebase

❌ Unable to rebase: user bbrowning is unknown.

Details

Please make sure bbrowning has logged in Mergify dashboard.

@bbrowning
Copy link
Contributor

@Mergifyio rebase

aakankshaduggal and others added 5 commits November 12, 2024 22:02
Signed-off-by: Aakanksha Duggal <[email protected]>
(cherry picked from commit 98c0fab)
Signed-off-by: Aakanksha Duggal <[email protected]>
(cherry picked from commit 5667ac9)
…g_format

Signed-off-by: Aakanksha Duggal <[email protected]>
(cherry picked from commit cd0b73a)
Signed-off-by: Aakanksha Duggal <[email protected]>
(cherry picked from commit 3040657)
Older versions of SDG did not have the `use_legacy_pretraining_format`
parameter, so this just removes that from the backported code.

Signed-off-by: Ben Browning <[email protected]>
@mergify
Copy link
Contributor Author

mergify bot commented Nov 12, 2024

rebase

✅ Branch has been successfully rebased

@bbrowning bbrowning force-pushed the mergify/bp/release-v0.3/pr-366 branch from d2a179a to 625d9ab Compare November 12, 2024 22:02
@mergify mergify bot removed the ci-failure label Nov 12, 2024
@bbrowning
Copy link
Contributor

I will need to investigate why mergify didn't auto-merge this, but it looks like just a gap in the auto-merge rules at least for this release branch. Merging manually since all tests passed and it has multiple approvals.

@bbrowning bbrowning merged commit fbfe7d4 into release-v0.3 Nov 13, 2024
18 checks passed
@bbrowning bbrowning deleted the mergify/bp/release-v0.3/pr-366 branch November 13, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration release-branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants