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

A lot of cleanup #64

Merged
merged 1 commit into from
Jul 26, 2023
Merged

A lot of cleanup #64

merged 1 commit into from
Jul 26, 2023

Conversation

kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Jul 26, 2023

  • Made a number of APIs not-nullable
  • Made a number of fields final where they could be
  • Added types to a few public APIs
  • Enabled and fixed a bunch of new lints
  • Moved part files to the lib/src dir
  • Require Dart 3

- Made a number of APIs not-nullable
- Made a number of fields final where they could be
- Added types to a few public APIs
- Enabled and fixed a bunch of new lints
- Moved part files to the lib/src dir
- Require Dart 3
@kevmoo
Copy link
Contributor Author

kevmoo commented Jul 26, 2023

Ooo...looks like CI might be turned off for your repo 🤷

@wstrange
Copy link
Owner

Hi Kev - this looks awesome. Not sure why the CI is not running... let me take a look

@wstrange
Copy link
Owner

I think GH disabled the workflow after 60 days. I'm going to merge this now (assuming you've run all tests/) - and I'll figure out how to enable CI later.

Thanks so much for this PR!

@wstrange wstrange merged commit bea77eb into wstrange:master Jul 26, 2023
@kevmoo kevmoo deleted the cleanup branch July 26, 2023 01:39
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