Skip to content

Null safety support#58

Open
babulpatel1309 wants to merge 1 commit into
pinkfish:mainfrom
babulpatel1309:main
Open

Null safety support#58
babulpatel1309 wants to merge 1 commit into
pinkfish:mainfrom
babulpatel1309:main

Conversation

@babulpatel1309

@babulpatel1309 babulpatel1309 commented May 19, 2021

Copy link
Copy Markdown
  • Added null safety and updated dependencies.

@pinkfish

Copy link
Copy Markdown
Owner

I did an update to null safety. Is this still needed?

@babulpatel1309

Copy link
Copy Markdown
Author

@pinkfish if everything is working as expected then we're good to go :)
The PR was actually generated on 19th May 🤣 . At that moment so many libraries were missing this part, so i sat and raised PR for all libraries.

@babulpatel1309

Copy link
Copy Markdown
Author

@pinkfish i coudn't able to find where exactly did you publish the null safety update?

@philipgiuliani

Copy link
Copy Markdown

@pinkfish you have updated to null safety in the master branch but not in the main branch.

@fotiDim

fotiDim commented Oct 30, 2021

Copy link
Copy Markdown

@philipgiuliani why is there both a master and a main branch? For sound null safety sdk: ">=2.12.0 <3.0.0" should be defined in pubspec.yaml. I see that this is the case only in master and not main.

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.

5 participants