pubspec icon indicating copy to clipboard operation
pubspec copied to clipboard

I've created and tested a null safe version.

Open bsutton opened this issue 4 years ago • 11 comments

It currently relies on an unpublished version of uri but it looks like its about to be published.

Obviously you don't want to merge until the uri version is published but since I needed a local copy I thought I should push the changes I've made.

bsutton avatar Feb 16 '21 23:02 bsutton

Whilst experimenting I added a funding.yml to this project. Only just noticed after raising the PR. I've now removed.

bsutton avatar Feb 16 '21 23:02 bsutton

And the null safe version of the uri package has just been plublished!

bsutton avatar Feb 17 '21 00:02 bsutton

Can someone review this? @aholmgren

j4qfrost avatar Feb 26 '21 06:02 j4qfrost

Time to merge this!

gmpassos avatar Mar 08 '21 08:03 gmpassos

I need to find a new owner for this repo as I no longer have anything to do with dart

any volunteers?

Andersmholmgren avatar Mar 09 '21 07:03 Andersmholmgren

@bsutton seems like a solid candidate and already has pubspec2 up on pubdev, but if he doesn't want to take up the mantle then I will, I am always online.

j4qfrost avatar Mar 09 '21 07:03 j4qfrost

@j4qfrost @Andersmholmgren I'm already over committed with a no. of other FOS projects so I will decline on this one.

@j4qfrost if you are happy to take up the mantel I will help where I can.

bsutton avatar Mar 09 '21 07:03 bsutton

Cool, I'll take over the project. Thanks for building it. I'll see if I can actually push out a nullsafety version tonight on pubdev.

Unless you want to @Andersmholmgren

j4qfrost avatar Mar 09 '21 07:03 j4qfrost

Thanks.

On Tue, 9 Mar 2021, 6:15 pm j4qfrost, [email protected] wrote:

Cool, I'll take over the project. Thanks for building it. I'll see if I can actually push out a nullsafety version tonight on pubdev.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Andersmholmgren/pubspec/pull/23#issuecomment-793475322, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG32OFXK4M65RIEBFJO6RLTCXDKDANCNFSM4XXJKLVA .

bsutton avatar Mar 09 '21 09:03 bsutton

It's all setup in my fork and published, I merged in the const constructors as well, sorry for the wait.

j4qfrost avatar Mar 15 '21 10:03 j4qfrost

@j4qfrost thanks for the work!

bsutton avatar Mar 19 '21 00:03 bsutton