Listable icon indicating copy to clipboard operation
Listable copied to clipboard

Add SwiftUILists pod

Open kyleve opened this issue 2 years ago • 4 comments

Getting the base targets set up for the SwiftUI interface. I am not including anything in the changelog yet.

kyleve avatar Nov 19 '23 23:11 kyleve

Would you be open to doing this on a feature branch until it's ready to be integrated upstream?

n8chur avatar Nov 20 '23 19:11 n8chur

Would you be open to doing this on a feature branch until it's ready to be integrated upstream?

We're not publishing the pod at all, so main seemed fine to me. This repo doesn't see much action anyway...

kyleve avatar Nov 20 '23 20:11 kyleve

I'd prefer this to go on a feature branch, but don't feel too strongly—maybe let's get one more review with a comment regarding this before merging?

  • [ ] 1 more review addressing feature branch topic

n8chur avatar Nov 20 '23 20:11 n8chur

So I think I've said this before, but maybe haven't written it down: Feature branches are only useful when, without one, you're going to be forced to ship something buggy or incomplete to your customers – but they come with downsides, eg the longer running they are, the more of a pain it is to merge back and forth, and avoid incongruent changes on main vs the-feature/branch. This is why instead of feature branches in POS, we tend to leverage feature flags for example. Given that we can just not publish this pod (or any other pods) until they're ready, the feature branch in general is just extra legwork for no benefit, IMO.

kyleve avatar Nov 20 '23 20:11 kyleve