Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dart null safety and some other changes #24

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ali-alizadeh
Copy link

Hello. Thank you @AdityaMulgundkar for this great plugin!

Dart null safety has been around some time. So I decided to migrate this plugin.
I have migrated the project to null safety using dart migrate command, and have made some edits on the changes made by it (Fixes #23).
I have also updated pedantic packages to the latest version and fixed new lint errors.

Let me know if there's something wrong.

Thanks.

@ali-alizadeh ali-alizadeh changed the title Dart null safety ans some other changes Dart null safety and some other changes Apr 16, 2021
Copy link

@Joldersman Joldersman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for rewriting the code to null safety.

Copy link

@OutdatedGuy OutdatedGuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍🏼

@ZahraVe
Copy link

ZahraVe commented Oct 8, 2021

@ali-alizadeh Thanks for migration, I am going to use this pckage, how can I get the null safe version in my flutter app?

@ali-alizadeh
Copy link
Author

@ZahraVe Sorry for the very late response.
My changes have not been merged and released yet.
Although you can use my fork directly from git; Or fork a version for yourself from mine.

...
dependencies:
  wallpaper_manager:
    git: https://github.com/ali-alizadeh/wallpaper_manager.git
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dart null safety
4 participants