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

Comment while we decide what to do #2085

Closed
wants to merge 1 commit into from
Closed

Conversation

mauromorales
Copy link
Member

This will no longer work with versioneer so I commented the test. I'm wondering if the test should do something else though or if we remove it completely

@mauromorales mauromorales requested a review from a team December 14, 2023 15:39
@mauromorales mauromorales self-assigned this Dec 14, 2023
Signed-off-by: Mauro Morales <mauro.morales@spectrocloud.com>
@jimmykarily
Copy link
Contributor

It's already been tested here: https://github.com/kairos-io/kairos-sdk/blob/74a57eb67370af042872fb67bcd07aa0ee5d02eb/versioneer/tag_list_test.go#L356 not sure if we want to test is through kairos-agent too to make sure versioneer is consumed correctly.

@mauromorales
Copy link
Member Author

we still want the test for integration testing, but we should move into another test so we don't waste time spining up #2081

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants