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

Eliminate variant in version comparison for takeover action #39

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

kb2ma
Copy link
Contributor

@kb2ma kb2ma commented Dec 13, 2024

This variant may be appended by balena-proxy.

Change-type: patch

@kb2ma kb2ma requested review from pipex and thgreasi December 13, 2024 16:25
@kb2ma
Copy link
Contributor Author

kb2ma commented Dec 13, 2024

Do we need to be concerned with the '.prod' variant as well? I have not seen this used.

@kb2ma kb2ma changed the title Eliminate '.dev' in version comparison for takeover action Eliminate '.dev' variant in version comparison for takeover action Dec 13, 2024
@kb2ma kb2ma force-pushed the strip-variant-comparison branch from 8ec3ba5 to 913a23f Compare December 13, 2024 16:42
@kb2ma
Copy link
Contributor Author

kb2ma commented Dec 13, 2024

Added a second commit that extends the first to check for .prod as well as .dev. I can either squash down if we need both tests, or remove the second commit if we don't.

@kb2ma kb2ma changed the title Eliminate '.dev' variant in version comparison for takeover action Eliminate variant in version comparison for takeover action Dec 13, 2024
lib/index.ts Outdated Show resolved Hide resolved
tests/01-actions.spec.ts Outdated Show resolved Hide resolved
@kb2ma kb2ma requested a review from thgreasi December 14, 2024 01:05
The variant may be appended by balena-proxy.

Change-type: patch
Signed-off-by: Ken Bannister <kb2ma@runbox.com>
@kb2ma kb2ma force-pushed the strip-variant-comparison branch from ce66988 to b0e0a62 Compare December 16, 2024 22:24
@kb2ma kb2ma marked this pull request as ready for review December 16, 2024 22:25
@flowzone-app flowzone-app bot merged commit 61febea into master Dec 16, 2024
55 checks passed
@flowzone-app flowzone-app bot deleted the strip-variant-comparison branch December 16, 2024 22:28
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