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

Kotest dependencies fixes #446

Merged
merged 6 commits into from
Sep 30, 2021
Merged

Conversation

Kantis
Copy link
Contributor

@Kantis Kantis commented Sep 28, 2021

🚀 Description

  1. Kotest extensions have independent lifecycles. Meaning the latest version of kotest-extensions-spring might be 2.0.0 while kotest-extensions-koin might still be at 1.1.0. Currently, these are bundled into a single versioning property.
  2. Added notation for two more kotest dependencies (Testing.kotest.framework.api for kotest-framework-api and Testing.kotest.framework.datatest for kotest-framework-datatest)

Please let me know if I missed something.

Copy link
Member

@LouisCAD LouisCAD left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.
There's an issue regarding your version key rules changes.
I'd recommend to rollback the changes into the txt files before implementing the fix and running the tests again.

@LouisCAD
Copy link
Member

There's no need for one rule per artifact. You can use stars, see the examples in other rules already in refreshVersions (linked from the contributing docs).

@Kantis
Copy link
Contributor Author

Kantis commented Sep 30, 2021

Ah, thank you! That's much nicer 😃

@LouisCAD
Copy link
Member

Thanks for the contribution!
Will check it locally soon.

@LouisCAD
Copy link
Member

It looks like the PR is not based on the main branch from 2 days ago as the PR submission would suggest, but it based on a commit from over 3 weeks ago 🤔

@LouisCAD
Copy link
Member

Also looks like you initially, correctly commited the removed-dependencies-versions-keys.txt file, but then wrongly removed it, while it's ciritical to have it to ensure automatic migration to the new version key, without upgrading anything.

Copy link
Member

@LouisCAD LouisCAD left a comment

Choose a reason for hiding this comment

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

Fixed what needed to be fixed.

Thanks!

@LouisCAD LouisCAD merged commit 432a48f into Splitties:main Sep 30, 2021
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