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

Re-create templates/app, add deprecation notices for app_shared and skeleton. #8336

Merged
merged 3 commits into from
Dec 26, 2024

Conversation

matanlurey
Copy link
Contributor

Part of flutter/flutter#160673, which removes --template skeleton.

This PR copies the templates/app_shared directory, which was only used to share code between templates/app and templates/skeleton, in preparation for removing skeleton (and making the existence of app_shared obsolete).

Issue tracking final cleanup: flutter/flutter#160692.

@ditman
Copy link
Member

ditman commented Dec 21, 2024

We need to update the CHANGELOG and pubspec.yaml to have this autopublished in pub.dev, so it can be consumed from the flutter tool.

The "Linux analyze" tests are failing because two new dart lints.

(I'll publish a temporary ignore for those to reopen the tree)

@ditman
Copy link
Member

ditman commented Dec 23, 2024

Working on the analysis failures here:

flutter/flutter#160799

Will land ASAP.

@@ -1,6 +1,7 @@
## NEXT
Copy link
Contributor

Choose a reason for hiding this comment

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

Without a version bump this won't be usable.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@matanlurey matanlurey force-pushed the copy-app-shared-as-app branch from 3e3675a to 1569466 Compare December 26, 2024 20:14
@matanlurey matanlurey dismissed stuartmorgan’s stale review December 26, 2024 20:46

Made requested change, thanks Stuart!

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 26, 2024
@auto-submit auto-submit bot merged commit 311b660 into main Dec 26, 2024
77 checks passed
@auto-submit auto-submit bot deleted the copy-app-shared-as-app branch December 26, 2024 20:46
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Jan 6, 2025
flutter/packages@eb73582...07ae98c

2025-01-06 49699333+dependabot[bot]@users.noreply.github.com
[dependabot]: Bump org.json:json from 20240303 to 20241224 in
/packages/in_app_purchase/in_app_purchase/example/android/app
(flutter/packages#8371)
2025-01-05 [email protected]
[google_maps_flutter] Activate leak testing (flutter/packages#8368)
2025-01-05 [email protected]
[flutter_markdown] Fix some memory leaks and activate leak testing
[prod-leak-fix] (flutter/packages#8367)
2025-01-03 [email protected] [image_picker]
Removes use of PHAsset on IOS 14+ (flutter/packages#8190)
2025-01-03 [email protected] Delete unused templates
`skeleton` and `app_shared` and release as `5.0.0`.
(flutter/packages#8360)
2025-01-02 [email protected] [tool] Ensure that
leak_tracker_flutter_testing is dev-only (flutter/packages#8365)
2025-01-02 [email protected] [pigeon] Discuss stability in README
(flutter/packages#8366)
2025-01-02 [email protected] [shared_preferences] Add information
about shared preferences android to docs (flutter/packages#8296)
2024-12-30 [email protected] [camera] Remove OCMock from
permission tests (flutter/packages#8350)
2024-12-29 [email protected] Drop vector bits from
allowed_unpinned_deps.yaml (flutter/packages#8327)
2024-12-28 [email protected] [camera] Fix
memory leaks in example and activate leak testing
(flutter/packages#8287)
2024-12-27 [email protected] [pigeon] Adds
platform for imports that aren't support on a platform
(flutter/packages#8338)
2024-12-26 [email protected] Bump Plugin Example Apps to
TargetSdkVersion >= 34 (flutter/packages#8285)
2024-12-26 [email protected] Re-create
`templates/app`, add deprecation notices for `app_shared` and
`skeleton`. (flutter/packages#8336)
2024-12-26 [email protected] Update Gradle Command Test to Only
Accept Gradle Declarative Apply (flutter/packages#8325)
2024-12-24 49699333+dependabot[bot]@users.noreply.github.com
[dependabot]: Bump com.google.guava:guava from 33.3.1-android to
33.4.0-android in /packages/camera/camera_android_camerax/android
(flutter/packages#8331)
2024-12-24 49699333+dependabot[bot]@users.noreply.github.com
[dependabot]: Bump camerax_version from 1.3.4 to 1.4.1 in
/packages/camera/camera_android_camerax/android (flutter/packages#8330)
2024-12-24 49699333+dependabot[bot]@users.noreply.github.com [gradle]:
Bump com.google.truth:truth, com.google.code.gson:gson,
com.squareup.okhttp3:okhttp in /packages/espresso/android
(flutter/packages#8089)
2024-12-24 49699333+dependabot[bot]@users.noreply.github.com
[image_picker]: Bump androidx.activity:activity from 1.9.2 to 1.9.3 in
/packages/image_picker/image_picker_android/android
(flutter/packages#7897)
2024-12-23 [email protected] [ci] Cumulative fixes to reopen the tree.
(flutter/packages#8344)
2024-12-18 [email protected] Group dependabot updates for some gradle
dependencies (flutter/packages#8100)
2024-12-18 [email protected] Applied Gradle Plugins Declaratively
All Remaining Example Apps (flutter/packages#8312)
2024-12-18 49699333+dependabot[bot]@users.noreply.github.com
[lifecycle]: Bump androidx.annotation:annotation from 1.7.0 to 1.9.1 in
/packages/flutter_plugin_android_lifecycle/android
(flutter/packages#7974)
2024-12-18 [email protected] [ci] Re-enable macOS sandboxing
(flutter/packages#8293)
2024-12-18 49699333+dependabot[bot]@users.noreply.github.com [espresso]:
Bump com.android.tools.build:gradle from 7.4.1 to 8.7.2 in
/packages/espresso/android (flutter/packages#8013)
2024-12-18 [email protected] [shared_preferences] Increase minimum
android endorsed version (flutter/packages#8318)
2024-12-17 [email protected] Revert "[shared_preferences] Add
shared preferences devtools" (flutter/packages#8314)
2024-12-16 [email protected] [shared_preferences] Add shared
preferences devtools (flutter/packages#6749)
2024-12-16 [email protected] [google_adsense] Add optional init
parameters. (flutter/packages#8297)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: flutter_template_images platform-web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants