kotools / types

Explicit types for Kotlin Multiplatform.
https://types.kotools.org
MIT License
90 stars 6 forks source link

🔥 Removal of `NotEmptyRange` and `*Bound` types #627

Closed augustomtt closed 8 months ago

augustomtt commented 8 months ago

Close #618

LVMVRQUXL commented 8 months ago

Thank you @augustomtt for your contribution! 🙏 I see that all your commits are signed right now. 😉 I will go throw the request and review it very soon. 😁

augustomtt commented 8 months ago

Sorry!! My mistake! I will fix it now 😬

augustomtt commented 8 months ago

So sorry for the mess I made, this are my first commits to open source repositories, it won't happen again! Could you fix the conflict? It seems related to a simple name-ordering in the changelog, it's up to you who goes first! 😅

LVMVRQUXL commented 8 months ago

No problem @augustomtt, I will help you on this! 😁

I'm gonna fix the commit history in a bit using the latest version of the main branch on this repository. My changes might force-push your branch. 😉

LVMVRQUXL commented 8 months ago

@augustomtt I am also editing your commit messages for better alignment with our conventions. 🧑‍💻

LVMVRQUXL commented 8 months ago

I've fixed the commit history @augustomtt and also added an entry in the unreleased changelog. 🙌 Let's review the changes while checks are running. 😉

augustomtt commented 8 months ago

Awesome!! 🤠

LVMVRQUXL commented 8 months ago

@augustomtt This request is successfully merged! 🎉

Feel free to show your support on this project by starring it on GitHub and activating the release notification for being noticed when your changes will be published to the world. 🌐

Also, if you have any feedback or suggestion about Kotools Types, I will be happy to read your thoughts. And as always, feel free to contribute on another issue or discussion. 😁