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

Added null safety support #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added null safety support #26

wants to merge 4 commits into from

Conversation

dpr10
Copy link

@dpr10 dpr10 commented Mar 18, 2021

  • updated example app
  • updated android embed version
  • migrate to androidX

  - updated android embed version
  - updated example app
  - used androidX on example app
@areille
Copy link
Contributor

areille commented Mar 23, 2021

Closes #23.

@glendmaatita
Copy link

Definitely need this

 - updated example
 - updated version according null safety migration guide
@dpr10
Copy link
Author

dpr10 commented Apr 5, 2021

PR updated. Thanks @friebetill

@mohamadhadibi
Copy link

please complete and merge this PR. thanks

@friebetill
Copy link

friebetill commented Apr 13, 2021

As far as I can see, the PR is complete. @KingWu just needs to look over it.

In the meantime you can already use the PR with the following in your pubspec.yaml:

dev_dependencies:
  ...
  # Waiting for this PR https://bit.ly/3tO37NG
  gen_lang:
    git:
      url: https://github.com/KingWu/gen_lang
      ref: 7ba7548
  ...

@Aina27
Copy link

Aina27 commented Apr 28, 2021

@KingWu we definitely need this, also to solve version issues , mainly with some packages depending on args

@maerlynflagg
Copy link

no activity of the author since september and last version May 2, 2020. :/ hopefully the activity will be changed to active.

@NicolasDionB
Copy link

Any progression on this? Please just approve it! @KingWu or someone else?

var parser = new ArgParser();

parser.addOption('source-dir',
defaultsTo: 'res/string',
callback: (String x) => i18nOption.sourceDir = x,
callback: (String? x) => i18nOption!.sourceDir = x,
Copy link
Contributor

Choose a reason for hiding this comment

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

if (I18nOption == null) this will throw an error

help: 'A source folder contains all string json files');

parser.addOption('output-dir',
defaultsTo: 'lib/generated',
callback: (String x) => i18nOption.outputDir = x,
callback: (String? x) => i18nOption!.outputDir = x,
Copy link
Contributor

Choose a reason for hiding this comment

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

if (I18nOption == null) this will throw an error

help: 'A output folder stores all generated files');

parser.addOption('template-locale',
defaultsTo: 'en',
callback: (String x) => i18nOption.templateLocale = x,
callback: (String? x) => i18nOption!.templateLocale = x,
Copy link
Contributor

Choose a reason for hiding this comment

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

if (I18nOption == null) this will throw an error

@NicolasDionB
Copy link

Now that Dart 3 forces the use of Null Safety, it would be a good time to complete this PR 😄

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.

9 participants