Skip to content

[ADFA-365] - project build test with no androidX Project - #8

Merged
romanlapa merged 1 commit into
stagefrom
feature/ADFA-365-no-androidx
Feb 14, 2025
Merged

[ADFA-365] - project build test with no androidX Project#8
romanlapa merged 1 commit into
stagefrom
feature/ADFA-365-no-androidx

Conversation

@YahorZCzygunki

Copy link
Copy Markdown
Contributor

No description provided.

@romanlapa
romanlapa merged commit ddb0ec9 into stage Feb 14, 2025
jatezzz pushed a commit that referenced this pull request Jun 22, 2026
[ADFA-365] - project build test with no androidX Project
fryanpan added a commit that referenced this pull request Jul 31, 2026
…ts (map commit #8)

Commits 1-7 moved every identifier; this moves the prose that describes them.
Zero identifier changes, zero behavior changes - 88 insertions against 88
deletions, every one of them inside a comment, a KDoc/javadoc block, an AIDL
javadoc, an XML comment, or a backticked test-method name.

Senses renamed

- "test app" / "test-app" / "test APK" -> "proxy app" / "proxy app APK", in the
  quickbuild-runtime AIDL javadoc, its AndroidManifest comments, its runtime
  javadoc, the daemon's Aapt2Link KDoc, and the runtime module's Gradle
  description. This is the installed app built from the generated proxy
  components, never the androidTest instrumentation APK.
- "setup build" / "setup-build time" -> "proxy app build" / "proxy app build
  time", across the same modules plus GradlePluginConfig's javadoc and the
  gradle-plugin sample project's comment.
- "fast path" (retired term) -> "live reload path", in the quick-build
  classifier/annotations/metrics-sink KDoc, QuickBuildMetrics, QuickBuildJson,
  and 14 backticked test-method names in ChangeClassifierTest and
  AnnotationImpactAnalyzerTest.
- "rebaseline-on-return hand-back" -> "refresh-baseline-on-return hand-back" in
  the Quick Build dropdown KDoc: that hand-back after a Standard Run build is
  RefreshBaseline, not a proxy app rebuild.

Two "fast path" occurrences named a different mechanism and were reworded to say
what they mean rather than to the new vocabulary: the daemon's Java-ABI
skip-recompile shortcut is now "the Java-ABI shortcut" (two test comments), and
AndroidProjectWatcher's inotify-vs-poll comment now says "the low-latency path".

Deliberately frozen

- The fastPathScope() / fastPathRoots identifiers and every KDoc link to them;
  only the prose around them moved.
- ensureBaseline (all 11 sites - it contains the substring "reBaseline"), the
  testAppId JSON key and its back-compat aliases, the setup.json filename and
  keys, CodeOnTheGo.qbnoseed.
- Bench and Firebase wire strings: "RequiresRebaseline", "Prewarming",
  "quick_build_rebaseline" and the comments that explain why each is pinned.
- Generic English "set up": JUnit setUp() in Aapt2LinkTest and
  IncrementalCompilerTest, "One-time session setup" on ConfigureRequest,
  "classpath from setup.json" in DaemonService.
- The androidTest sense of "test": TestAppCompatActivity, testAppend.
- quick-build/corpus/ and the historical dated docs.
- "fail-fast path" in ProxyAppInstaller - a different word that an -i grep for
  "fast path" still hits.

Verification (JUnit XML, not the BUILD SUCCESSFUL line - the root build sets
ignoreFailures=true): :quickbuild-runtime:test 552/0, :quickbuild-daemon:test
102/0, :quick-build:test 3510/0, :app:testV8DebugUnitTest 196/0.
:gradle-plugin:test 99 tests, 3 failures - the pre-existing AndroidIDEPluginTest
logsender trio, unrelated to this change; every quickbuild test passes.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QTVTzgiPq4evYCuPLTNUvL
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