Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STS vBump to 4.11.1.1 #25396

Merged
merged 1 commit into from
Feb 23, 2024
Merged

STS vBump to 4.11.1.1 #25396

merged 1 commit into from
Feb 23, 2024

Conversation

ssreerama
Copy link
Contributor

This PR adds a fix to the DacFx net8.0 version causing issues.

image

Copy link
Contributor

@kisantia kisantia left a comment

Choose a reason for hiding this comment

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

@ssreerama also please link the 2 issues this fixes

@coveralls
Copy link

Pull Request Test Coverage Report for Build 8023008855

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.001%) to 41.757%

Totals Coverage Status
Change from base Build 8014175898: 0.001%
Covered Lines: 30827
Relevant Lines: 69082

💛 - Coveralls

@Charles-Gagnon
Copy link
Contributor

@ssreerama And you've tested this locally to verify the issues are resolved with this bump? Want to avoid having to go through another round of PRs/Port requests if there's any other issues that come up

@ssreerama
Copy link
Contributor Author

@Charles-Gagnon , yes I have done end-end basic testing for sc and dacpac and dont see these issues or any other issues either.

@kisantia
Copy link
Contributor

checks for main completed and all passed, so merging this now to get RC build started sooner.

@kisantia kisantia merged commit 670de98 into release/1.48 Feb 23, 2024
10 of 14 checks passed
@kisantia kisantia deleted the sai/relStsVbump branch February 23, 2024 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants