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

pg: don't hard code major versions in upgrade_check task #249

Merged
merged 1 commit into from
Jan 13, 2022

Conversation

carobme
Copy link
Contributor

@carobme carobme commented Jan 10, 2022

Instead of hard coding the supported versions just make sure the version
consists of digits only. The API will not accept invalid versions anyway.
This will allow performing the upgrade_check for - and subsequently
upgrading to - pg14 (and upcoming versions, as soon as they are offered).

Copy link
Contributor

@laysauchoa laysauchoa left a comment

Choose a reason for hiding this comment

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

question 🙋🏻‍♀️

rominf
rominf previously requested changes Jan 13, 2022
Copy link
Contributor

@rominf rominf left a comment

Choose a reason for hiding this comment

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

@carobme The commit/PR messages are misleading. The command that you've fixed is not about performing an upgrade; it's for checking the possibility of it. Could you please adjust the messages?

Instead of hard coding the supported versions just make sure the version
consists of digits only. The API will not accept invalid versions anyway.
This will allow performing the upgrade_check for - and subsequently
upgrading to - pg14 (and upcoming versions, as soon as they are offered).
@carobme carobme requested a review from a team as a code owner January 13, 2022 12:00
@carobme carobme changed the title pg: allow upgrading to version 14 pg: don't hard code major versions in upgrade_check task Jan 13, 2022
@carobme carobme dismissed rominf’s stale review January 13, 2022 12:06

PR/commit message changed

@rominf rominf merged commit 3c9da94 into master Jan 13, 2022
@rominf rominf deleted the carobme-pg-versions branch January 13, 2022 12:09
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.

None yet

3 participants