flutter-studio / flutter-icons

Customizable Icons for Flutter :boom:
Apache License 2.0
332 stars 123 forks source link

Migrate to sound null safety #41

Closed adarsh-technocrat closed 3 years ago

adarsh-technocrat commented 3 years ago

Description

rgr commented 3 years ago

Any news ?

adarsh-technocrat commented 3 years ago

Any news ?

yup! there is an attached PR that will resolve this issue :)

rgr commented 3 years ago

Great news ! Thanks @adarsh-technocrat. Hope it gets accepted soon!

rgr commented 3 years ago

@2534290808 could you accept the null safety PR please?

joaovvrodrigues commented 3 years ago

@adarsh-technocrat how add this in my project ?

flutter_icons: git: url: https://github.com/adarsh-technocrat/flutter-icons/tree/null-safety-migration

don't work here, Could you upload your changes to your project's main?

adarsh-technocrat commented 3 years ago

flutter_icons

Hey, @joaovvrodrigues I would prefer you to make a different branch for your null safety version! here I will change the base branch to that null safety branch and then you can merge it =)

rgr commented 3 years ago

@adarsh-technocrat while waiting for your PR to be accepted did you succeed in importing your own branch into your project ? How do you do that ?

adarsh-technocrat commented 3 years ago

@adarsh-technocrat while waiting for your PR to be accepted did you succeed in importing your own branch into your project ? How do you do that ?

Sorry @LearningRaph I could not get you but as far as I could understand I just went through the series of steps for migrating this sample and package :-

  1. Forked the official repo
  2. Cloned the forked repo to my local machine
  3. Created the different branch for the null safety migration eg here :- git branch null-safety-migration
  4. Applied Migration in the null-safety-migration branch
  5. Committed the changes
  6. Pushed the branch to my forked repo
  7. Finally Generated the PR.

    Im not Sure @LearningRaph is this is the right Answer for your Question!

rgr commented 3 years ago

Nope :) I was wondering if you could use your fork in your flutter app. If we don't have news from the creator it would be a good option...

adarsh-technocrat commented 3 years ago

Ahh.. no @LearningRaph until and unless it is Publish to the pub. div I think we cannot use the null-safety version of this package :)

joaovvrodrigues commented 3 years ago

@adarsh-technocrat @LearningRaph in fact you can, and that's what I'm trying to say, If you move your changes to the project's Branch Master, you can use the package directly from git.

flutter_icons: git: url: https://github.com/adarsh-technocrat/flutter-icons

Captura de Tela 2021-03-06 às 14 47 04

because you created a new branch, then we can't access it, it needs to be on the master, as far as I know.

adarsh-technocrat commented 3 years ago

@adarsh-technocrat @LearningRaph in fact you can, and that's what I'm trying to say, If you move your changes to the project's Branch Master, you can use the package directly from git.

flutter_icons: git: url: https://github.com/adarsh-technocrat/flutter-icons

Captura de Tela 2021-03-06 às 14 47 04

because you created a new branch, then we can't access it, it needs to be on the master, as far as I know.

oooh got it @joaovvrodrigues thanks I'll do the changes right now =)

adarsh-technocrat commented 3 years ago

Hey, @joaovvrodrigues I merged the null-safety-migration branch to the master branch see now if you can access the null safety version!

joaovvrodrigues commented 3 years ago

It worked just fine! 🥇

rgr commented 3 years ago

Works for me too ! Great team work :)

ghost commented 3 years ago

As an info

some_package:
  git:
    url: blabla
    ref: some_branch

works to use a certain branch of a project instead of master

aytunch commented 3 years ago

@flutter-studio or @2534290808 Are you planning on accepting @adarsh-technocrat 's Null-safety PR? Also are you planning on maintaining this package? Or are you considering accepting new maintainers?

eric-taix commented 3 years ago

Until this PR has been merged to master and the package has been published to pub.dev, you can update your pubspec.yaml to:

  flutter_icons:
    git:
      url: https://github.com/adarsh-technocrat/flutter-icons.git
      branch: null-safety-migration
davidAg9 commented 3 years ago

any one tried to contact this package owner ...it will be great If they left a number /email or WhatsApp so we can contact them...it seem this and other packages I have used in my app are tending towards the possibility of being abandoned or not maintain or worse discontinued

davidAg9 commented 3 years ago

Until this PR has been merged to master and the package has been published to pub.dev, you can update your pubspec.yaml to:

  flutter_icons:
    git:
      url: https://github.com/adarsh-technocrat/flutter-icons.git
      branch: null-safety-migration

your package need null safety support too (fluid bottom nav bar)

aytunch commented 3 years ago

It is April, will this package be maintained in the future? We are depending on it a lot. If not, we will try to migrate to another solution unfortunately :/

joaovvrodrigues commented 3 years ago

@aytunch I agree with you, I believe that I will take the same action. There are similar packages that have already been migrated and are constantly being updated. Last update of the package was in June 2020.

aytunch commented 3 years ago

@aytunch I agree with you, I believe that I will take the same action. There are similar packages that have already been migrated and are constantly being updated. Last update of the package was in June 2020.

@joaovvrodrigues i hear you. Would you please share some of the alternatives which are being maintained and support null-safety so others can benefit as well

joaovvrodrigues commented 3 years ago

I believe that this package here will meet this need: https://pub.dev/packages/fluttericon

I think I will already start the migration progress in my app. It may not have exactly the same icons, but it always gets one that can be replaced by something similar.

ydang204 commented 3 years ago

Any news? I am waiting for this part too!

deadsoul44 commented 3 years ago

Any update?

komkumar commented 3 years ago

Any updates on migration to null safety?

fernando-s97 commented 3 years ago

Any updates?

joaovvrodrigues commented 3 years ago

@2534290808 , please merge this.

mrripp3r commented 3 years ago

Any news on migration to null safety?

hanyska commented 3 years ago

+1

diegopq commented 3 years ago

@udiedrichsen any news?

fernando-s97 commented 3 years ago

Why was it closed?

giorgio79 commented 3 years ago

https://pub.dev/packages/flutter_vector_icons PS This package looks like needs an IconButton, instead of the simple Icon

nehal076 commented 3 years ago

Any update ? Update to null safety.

aytunch commented 3 years ago

@adarsh-technocrat I am using your fork after null-safety but there is a dependency issue as you can see below.

Because flutter_native_splash >=1.1.9 depends on args ^2.1.1 and flutter_launcher_icons 0.9.0 depends on args 2.0.0, flutter_native_splash >=1.1.9 is incompatible with flutter_launcher_icons 0.9.0.

And because no versions of flutter_launcher_icons match >0.9.0 <0.10.0, flutter_native_splash >=1.1.9 is incompatible with flutter_launcher_icons ^0.9.0. So, because bare_bones depends on both flutter_launcher_icons ^0.9.0 and flutter_native_splash ^1.2.0, version solving failed.

pub get failed (1; So, because bare_bones depends on both flutter_launcher_icons ^0.9.0 and flutter_native_splash ^1.2.0, version solving failed.) exit code 1

luohao123 commented 1 year ago

Deprecate this lib , avvoid using it any longer, I have removed all icons from this lib, using FluentIcons instead.