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

sonarr: add support for snowflake and db2 (DAT-17397) #199

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Conversation

StevenMassaro
Copy link
Contributor

No description provided.

@StevenMassaro
Copy link
Contributor Author

Should I be incrementing the version on this PR?

@sayaliM0412
Copy link
Contributor

Should I be incrementing the version on this PR?

is the change for this tested ?

@StevenMassaro
Copy link
Contributor Author

Should I be incrementing the version on this PR?

is the change for this tested ?

No. Not sure how to test it, to be honest. But since it follows the existing pattern with the other databases, I thought it seemed logical. Here's the PRs where it will be implemented (but of course, these PRs depend on this being merged first):

liquibase/liquibase#5837

https://github.com/liquibase/liquibase-pro/pull/1670

@sayaliM0412
Copy link
Contributor

Should I be incrementing the version on this PR?

is the change for this tested ?

No. Not sure how to test it, to be honest. But since it follows the existing pattern with the other databases, I thought it seemed logical. Here's the PRs where it will be implemented (but of course, these PRs depend on this being merged first):

liquibase/liquibase#5837

liquibase/liquibase-pro#1670

you can do something like this to test if it works and so that we can upgrade the build-logic tag later ones we know it's tested and works:
uses: liquibase/build-logic/.github/workflows/sonar-test-scan.yml@DAT-17397

over here: https://github.com/liquibase/liquibase/blob/c2f1a29086ab0e14861a580c4f979e2b1972d5be/.github/workflows/run-tests.yml#L313

@StevenMassaro
Copy link
Contributor Author

@sayaliM0412 I tested this code here: https://github.com/liquibase/liquibase/actions/runs/8806319321/job/24171805932

@jandroav I made those changes to Liquibase here: liquibase/liquibase#5837

Though I added Snowflake support in this PR, we're not actually ready to use it in OSS or Pro yet. I did log https://datical.atlassian.net/browse/DAT-17537 to fix that.

@StevenMassaro StevenMassaro changed the title sonarr: add support for snowflake and db2 sonarr: add support for snowflake and db2 (DAT-17397) Apr 24, 2024
….7.3

build(extension-attach-artifact-release.yml): update liquibase/build-logic version to v0.7.3
build(extension-release-prepare.yml): update liquibase/build-logic version to v0.7.3
build(extension-release-published.yml): update liquibase/build-logic version to v0.7.3
build(fossa_ai.yml): update liquibase/build-logic version to v0.7.3
build(os-extension-test.yml): update liquibase/build-logic version to v0.7.3
build(pom-release-published.yml): update liquibase/build-logic version to v0.7.3
build(pro-extension-test.yml): update liquibase/build-logic version to v0.7.3
docs(README.md): update build-logic version to v0.7.3 in documentation
@jandroav
Copy link
Contributor

Nice @StevenMassaro, please change @v0.7.0 to @v0.7.3 (the future build-logic release) in liquibase/liquibase#5837

I am updating versions in this PR: 16ba11f

@StevenMassaro
Copy link
Contributor Author

Thanks @jandroav ! @jnewton03 or @sayaliM0412 , could we get a review and a merge on this story?

@sayaliM0412 sayaliM0412 merged commit 67d43e2 into main Apr 24, 2024
@sayaliM0412 sayaliM0412 deleted the DAT-17397 branch April 24, 2024 13:43
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.

4 participants