Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Sep 25, 2021

Regenerate output
Also put in a pointer to the json_serializable issue related to creating
stand-alone enum converters

Regenerate output
Also put in a pointer to the json_serializable issue related to creating
stand-alone enum converters
@cyanglaz
Copy link
Contributor

LGTM, we need to do a patch version bump, and add a new entry to the CHANGELOG.

@kevmoo
Copy link
Contributor Author

kevmoo commented Sep 26, 2021

@cyanglaz – there is zero runtime affect here, though.

Do you always release every tweaks to the code?

@cyanglaz
Copy link
Contributor

@kevmoo we usually release what makes sense. If it is ok to not release this version, you can add a NEXT entry in the CHANGELOG, so it will be included in the next release. See https://github.com/flutter/plugins/blob/af2896b199ecc3cc5e1ed3000b009f65fc05c9cd/packages/wifi_info_flutter/wifi_info_flutter/CHANGELOG.md#next

@kevmoo
Copy link
Contributor Author

kevmoo commented Sep 27, 2021

@cyanglaz – PTAL

Co-authored-by: Chris Yang <ychris@google.com>
@kevmoo
Copy link
Contributor Author

kevmoo commented Sep 27, 2021

@cyanglaz – PTAL

Copy link
Contributor

@cyanglaz cyanglaz left a comment

Choose a reason for hiding this comment

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

LGTM

@cyanglaz cyanglaz added the waiting for tree to go green (Use "autosubmit") This PR is approved and tested, but waiting for the tree to be green to land. label Sep 27, 2021
@kevmoo kevmoo merged commit 5ec6196 into master Sep 27, 2021
@kevmoo
Copy link
Contributor Author

kevmoo commented Sep 27, 2021

Ooops! Just saw the label, @cyanglaz – did I break something?

@cyanglaz
Copy link
Contributor

It should be fine, the label is just for auto-merge :) You can still do it manually after ensuring the ci passes.

fluttergithubbot pushed a commit to flutter/flutter that referenced this pull request Sep 27, 2021
NickalasB added a commit to NickalasB/plugins that referenced this pull request Sep 27, 2021
* master:
  Load navigation controls immediately. (flutter#4378)
  [ci] Temporary run publish task on Flutter stable channel. (flutter#4388)
  [in_app_purchase] Bump dependencies on json_serializable, build_runner (flutter#4386)

# Conflicts:
#	packages/webview_flutter/webview_flutter_wkwebview/CHANGELOG.md
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Oct 29, 2021
amantoux pushed a commit to amantoux/plugins that referenced this pull request Dec 11, 2021
flutter#4386)

Regenerate output
Also put in a pointer to the json_serializable issue related to creating
stand-alone enum converters

Co-authored-by: Chris Yang <ychris@google.com>
KyleFin pushed a commit to KyleFin/plugins that referenced this pull request Dec 21, 2021
flutter#4386)

Regenerate output
Also put in a pointer to the json_serializable issue related to creating
stand-alone enum converters

Co-authored-by: Chris Yang <ychris@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes p: in_app_purchase platform-android platform-ios waiting for tree to go green (Use "autosubmit") This PR is approved and tested, but waiting for the tree to be green to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants