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

Fix textTheme deprecation and null safety issue #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maheshj01
Copy link

@maheshj01 maheshj01 commented May 14, 2024

@maheshj01
Copy link
Author

maheshj01 commented May 14, 2024

Anyone awaiting for this fix to be merged you can add this to your pubspec.yaml

  feature_discovery:
    git:
      url: https://github.com/maheshj01/feature_discovery.git
      ref: master

@deakjahn
Copy link

Thanks but considering that there are almost a dozen outstanding PRs... Sad to see this abandoned.

@TheFe91
Copy link

TheFe91 commented May 30, 2024

@maheshmnj you should consider to publish your version to pub.dev with a new name maybe

@maheshj01
Copy link
Author

@TheFe91 I don't think it would be right to publish someone else's work. Also, I can't promise any further updates to the package. But I can ensure this keeps running on future versions of flutter https://github.com/maheshmnj/feature_discovery.git

@richdreamalytics
Copy link

@ maheshj01 the packages has an MIT license, so it's fine republish, but I take your point about not being able to guarantee future support.

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