Skip to content

Conversation

@agniswarmandal
Copy link

Updated Library to flutter 2.8

@Airboy
Copy link

Airboy commented Feb 18, 2022

+1 for this one. That would be nice to merge it asap 🙏

@TianyongWang
Copy link

TianyongWang commented Feb 18, 2022 via email

@Airboy
Copy link

Airboy commented Apr 6, 2022

Any update on this @hurshi @agniswarmandal ?

flutter:
sdk: flutter
json_serializable: ^4.1.4
json_serializable: ^6.1.4
Copy link

@clragon clragon May 1, 2022

Choose a reason for hiding this comment

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

json_serializable should be in dev_dependencies not the normal dependencies,
because its not needed when compiling.

this will also prevent conflicts in the future.

Copy link

Choose a reason for hiding this comment

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

@agniswarmandal can you update this PR based on @clragon's comment?

@TianyongWang
Copy link

TianyongWang commented May 1, 2022 via email

@Airboy
Copy link

Airboy commented May 4, 2022

@hurshi this PR looks LGTM. Can you merge it?
Do you need any help or support to maintain this project?

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.

4 participants