search

xmolecules / jmolecules

Libraries to help developers express architectural abstractions in Java code
http://jmolecules.org
Apache License 2.0
1.26k stars 105 forks source link

`ValueObject` interface for KMolecules #100

Closed NathanCAllen closed 1 year ago

NathanCAllen commented 1 year ago

Since #51 was implemented before #55, ValueObject was never added as an interface in KMolecules.

I'd create the PR for this myself, but I don't have write access to the repo.

denniseffing commented 1 year ago

I created a PR for this issue as I just stumbled upon this as well.

https://github.com/xmolecules/jmolecules/pull/101

@NathanCAllen You don't need write access to a repository to create a PR, you can fork the repository, push your changes on a branch and then open up a PR in the original repository.

odrotbohm commented 1 year ago

That's merged, thanks!

mstine commented 1 year ago

Possible to get a release containing this fix quickly? I'm unfortunately unable to pull external snapshot dependencies.

odrotbohm commented 1 year ago

Kotlin? Have you switched to the dark side, Matt? 😜 I can wrap up a release tomorrow (Friday).

mstine commented 1 year ago

Kotlin is the dark side? 😂 At first glance I thought you were teasing me about my snapshots policy. At any rate, I've been writing pretty much 100% Kotlin since mid-2020. Oops.

Also, thanks! I've been following JMolecules from afar for awhile, but I'm finally getting a chance to use it!

odrotbohm commented 1 year ago

No package scope -> dark side. A hill I am wiling to die on. 😬

mstine commented 1 year ago

You're not wrong. I don't like it either. But cost/benefit to me has worked out pretty well.