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.

Sign up for GitHub

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

Jump to bottom

[pigeon] requires analyzer 5.2.0 #3090

Merged
merged 4 commits into from
Jan 25, 2023
Merged

[pigeon] requires analyzer 5.2.0 #3090

merged 4 commits into from
Jan 25, 2023

Conversation

tarrinneal
Copy link
Contributor

fixes flutter/flutter#118310 (comment)

Updates analyzer to 5.0.0 and updates deprecated name2 uses to name.

stuartmorgan
stuartmorgan approved these changes Jan 25, 2023
View reviewed changes
Copy link
Contributor

@stuartmorgan stuartmorgan left a comment

Choose a reason for hiding this comment

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

LGTM. Too bad we can't still support 4.x, but it'll work it's way through the ecosystem.

packages/pigeon/pubspec.yaml Outdated Show resolved Hide resolved
@tarrinneal tarrinneal added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 25, 2023
@tarrinneal tarrinneal changed the title [pigeon] requires analyzer 5.0.0 [pigeon] requires analyzer 5.2.0 Jan 25, 2023
@auto-submit auto-submit bot merged commit 59707ba into flutter:main Jan 25, 2023
sybrands-place pushed a commit to sybrands-place/packages that referenced this pull request Jan 30, 2023
Merge branch 'main' into null_status_retriable
577bbb1 
* main: (479 commits)
  removes raw ArrayLists (flutter#3101)
  Roll Flutter from c9affdb to 27f8ebd (15 revisions) (flutter#3098)
  [ci] Fix the new LUCI iOS build-all tasks (flutter#3099)
  [pigeon] [ObjC] Removes unused GetNullableObject function (flutter#3100)
  [pigeon] Swift host error handling (flutter#3084)
  Roll Flutter from a815ee6 to c9affdb (23 revisions) (flutter#3093)
  [ci] Enable min SDK version checks (flutter#3095)
  [pigeon] Fix C++ config handling (flutter#3094)
  [ci] Add LUCI version of iOS build-all (flutter#3096)
  [pigeon] Adds SwiftFunction annotation (flutter#2304)
  [flutter_adaptive_scaffold] Change `selectedIndex` on `standardNavigationRail` to allow null value. (flutter#3088)
  [pigeon] requires analyzer 5.2.0 (flutter#3090)
  Roll Flutter from c35efda to a815ee6 (22 revisions) (flutter#3089)
  [ci] Update legacy Flutter version tests (flutter#3087)
  Roll Flutter (stable) from 135454af3247 to b06b8b271095 (2551 revisions) (flutter#3086)
  [flutter_adaptive_scaffold] Fix leading and trailing Navigation Rail Widgets (flutter#3080)
  Roll Flutter from bd7bee0 to c35efda (24 revisions) (flutter#3085)
  [pigeon] Minor C++ output adjustments (flutter#3083)
  [pigeon] Updates writeScoped and addScoped to disallow symbol-less use. (flutter#3081)
  Roll Flutter from f33e8d3 to bd7bee0 (5 revisions) (flutter#3082)
  ...
Maatteogekko pushed a commit to Maatteogekko/packages that referenced this pull request Feb 4, 2023
@tarrinneal @Maatteogekko
[pigeon] requires analyzer 5.2.0 (flutter#3090)
6434fc5 
* analyzer 5.0.0

* fix changelog

* 5.2

* ^
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stuartmorgan stuartmorgan stuartmorgan approved these changes

@gaaclarke gaaclarke Awaiting requested review from gaaclarke

Assignees
No one assigned
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: pigeon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Pigeon] enable support Analyzer > 5.0.0
2 participants
@tarrinneal @stuartmorgan