refactor(device_info_plus)!: two-package federated architecture

[nohli]

Description

Refactor device_info_plus from the federated architecture to a platform interface and a package with all platform implementations.

Related Issues

#1226

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I titled the PR using Conventional Commits.
• I did not modify the CHANGELOG.md nor the pubspec.yaml files.
• All existing and new tests are passing.
• The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

• Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
• No, this is not a breaking change.

[nohli]

@miquelbeltran could you please check the web build (can easily be reproduced locally when running the example app in the browser)?
Do we need a conditional import/export of some sort for ffi/the windows implementation?

[miquelbeltran]

I will take a look!

[miquelbeltran]

@nohli seems that a conditional export did the trick!

[nohli]

@nohli seems that a conditional export did the trick!

Nice, so my dev intuition was right 😄

Thanks for trying and fixing! ❤️

[miquelbeltran]

LGTM! just a couple of questions

[vbuberen]

I meant not just one line, but return info that was in podspec before 😄

[nohli]

@miquelbeltran what about this issue? Should we say he can apply the changes to the latest architecture?

(If we close it unmerged, does it still count towards hacktoberfest?)

[vbuberen]

(If we close it unmerged, does it still count towards hacktoberfest?)

It is not.

Should we say he can apply the changes to the latest architecture?

If he agrees to do so.

[miquelbeltran]

I believe it counts because it has the labels.

[nohli]

If we want the change, we should communicate it in the other PR. Otherwise - if it's not updated - it can't be merged any more due to the new architecture.

Btw. could you also give my 5 refactoring PRs the labels?

[miquelbeltran]

I will just fix the merge conflict and go ahead with it, I still like the idea of removing the toMap method eventually