Skip to content

Cleanup all the cruft from being a plugin, now we are a dart package with hooks - #264

Merged
HamdaanAliQuatil merged 4 commits into
google:masterfrom
jonasfj:cleanup-plugin
May 1, 2026
Merged

HamdaanAliQuatil merged 4 commits into
google:masterfrom
jonasfj:cleanup-plugin

Conversation

@jonasfj

@jonasfj jonasfj commented Apr 30, 2026

Copy link
Copy Markdown
Member

This is an attempt to remove all the plugin left-over and rely exclusively on native hooks.

I had to re-create the example/, because the templates change so much that I couldn't keep up.

everything in example/ is boilerplate flutter except:

  • lib/main.dart (copied from the old example)
  • integration_test/
  • pubspec.yaml

@jonasfj

jonasfj commented Apr 30, 2026

Copy link
Copy Markdown
Member Author

wow, I did not think I was likely to get CI passing, after all the files I just deleted 🙈

Comment thread pubspec.yaml Outdated
Comment thread tool/test.sh Outdated
@jonasfj

jonasfj commented May 1, 2026

Copy link
Copy Markdown
Member Author

maybe we should disable firefox testing on mac os, it seems to fail a bit intermittently.

And honestly, testing firefox on Linux is probably good enough.

Co-authored-by: HamdaanAliQuatil <96776914+HamdaanAliQuatil@users.noreply.github.com>

@HamdaanAliQuatil HamdaanAliQuatil left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good!
Nice to finally have webcrypto as a Dart package with hooks :)

@HamdaanAliQuatil
HamdaanAliQuatil merged commit 9597502 into google:master May 1, 2026
16 of 18 checks passed
@jonasfj
jonasfj deleted the cleanup-plugin branch May 1, 2026 17:05
@mosuem mosuem mentioned this pull request Jul 17, 2026
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.

2 participants