Skip to content

Fix stale artifact IDs in quick-start docs#179

Merged
kirich1409 merged 1 commit intomainfrom
docs/verify-quickstart
Apr 29, 2026
Merged

Fix stale artifact IDs in quick-start docs#179
kirich1409 merged 1 commit intomainfrom
docs/verify-quickstart

Conversation

@kirich1409
Copy link
Copy Markdown
Contributor

Summary

  • README.md and docs/getting-started.md both referenced incorrect Maven artifact coordinates that were missing the featured- prefix
  • Fixed four artifact IDs: corefeatured-core, datastore-providerfeatured-datastore-provider, sharedpreferences-providerfeatured-sharedpreferences-provider, firebase-providerfeatured-firebase-provider
  • Fixed Maven Central badge URL in README.md (same corefeatured-core mismatch)
  • All other quick-start content verified against source: class/function names, ConfigValues constructor, FlagRegistry.register(), FeatureFlagsDebugScreen, SharedPreferencesProviderConfig, SPM product name FeaturedCore, iOS xcconfig steps, Gradle plugin ID — all accurate

Note on task brief: The brief stated "defaultLocalProvider() on JVM now returns JavaPreferencesConfigValueProvider (just merged)". The actual origin/main source for JVM still returns InMemoryConfigValueProvider with a TODO(#66) comment. No docs reference JVM provider behaviour in the quick-start, so there is nothing to fix there — but the task description appears to describe work that has not yet landed on main.

Test plan

  • Verify artifact coordinates resolve on Maven Central: dev.androidbroadcast.featured:featured-core, featured-datastore-provider, featured-sharedpreferences-provider, featured-firebase-provider

Closes #161

🤖 Generated with Claude Code

core/datastore/sharedpreferences/firebase provider coords were missing
the `featured-` prefix; Maven Central badge artifact also corrected.

Closes #161
@qodo-code-review
Copy link
Copy Markdown

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

@kirich1409 kirich1409 merged commit ef6b72a into main Apr 29, 2026
9 of 10 checks passed
@kirich1409 kirich1409 deleted the docs/verify-quickstart branch April 29, 2026 20:18
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.

docs: verify quick-start guide accuracy

1 participant