Skip to content

update dependencies #578

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

Merged
merged 1 commit into from
May 5, 2025
Merged

update dependencies #578

merged 1 commit into from
May 5, 2025

Conversation

shueybubbles
Copy link
Collaborator

Modern SQL, modern Go, and misc updates for packages with CVEs.

Copy link

@Copilot 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 upgrades the SQL server docker images from SQL 2017 to SQL 2022 to address dependency and CVE updates along with miscellaneous package updates.

  • Updated the docker image and related parameters in the pipeline configuration.
  • Modified the workflow docker run command to use the SQL 2022 image.

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

File Description
.pipelines/TestSql2017.yml Updated display names, arguments, and RunName from SQL2017 to SQL2022. Consider if the file name should also be updated to reflect SQL 2022.
.github/workflows/pr-validation.yml Updated docker run command to use SQL 2022 image.
Files not reviewed (2)
  • .vscode/settings.json: Language not supported
  • go.mod: Language not supported
Comments suppressed due to low confidence (1)

.pipelines/TestSql2017.yml:10

  • [nitpick] The file name '.pipelines/TestSql2017.yml' is now misleading given that the pipeline runs a SQL 2022 image. Consider renaming the file to reflect the updated SQL version.
steps:

Copy link
Contributor

@apoorvdeshmukh apoorvdeshmukh left a comment

Choose a reason for hiding this comment

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

:shipit:

@shueybubbles shueybubbles merged commit d3948b9 into main May 5, 2025
6 checks passed
@shueybubbles shueybubbles deleted the shueybubbles/updates branch May 5, 2025 15:07
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.

2 participants