Skip to content

refactor(geolocation): rename availabilitySignal to availabilityHintSignal#24276

Merged
mcollovati merged 1 commit intomainfrom
geo-avail-signal
May 8, 2026
Merged

refactor(geolocation): rename availabilitySignal to availabilityHintSignal#24276
mcollovati merged 1 commit intomainfrom
geo-avail-signal

Conversation

@Artur-
Copy link
Copy Markdown
Member

@Artur- Artur- commented May 6, 2026

Renames the per-UI availability accessor and slims the surrounding JavaDoc. The "Hint" suffix sets expectations about the signal's best-effort nature: Safari never reports permission state, Firefox does not always propagate browser-settings changes, and Chromium has a small propagation delay. The slimmer JavaDoc keeps a short reliability list in place of the previous multi-paragraph caveats section.

@github-actions github-actions Bot added the +1.0.0 label May 6, 2026
…ignal

Renames the per-UI availability accessor and slims the surrounding
JavaDoc. The "Hint" suffix sets expectations about the signal's
best-effort nature: Safari never reports permission state, Firefox does
not always propagate browser-settings changes, and Chromium has a small
propagation delay. The slimmer JavaDoc keeps a short reliability list
in place of the previous multi-paragraph caveats section.
@Artur- Artur- force-pushed the geo-avail-signal branch from 8596842 to f40148b Compare May 8, 2026 08:46
@Artur- Artur- requested a review from mcollovati May 8, 2026 08:47
@Artur- Artur- marked this pull request as ready for review May 8, 2026 08:47
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Test Results

 1 404 files  ±0   1 404 suites  ±0   1h 20m 45s ⏱️ - 1m 4s
10 128 tests ±0  10 058 ✅ ±0  70 💤 ±0  0 ❌ ±0 
10 603 runs  ±0  10 524 ✅ ±0  79 💤 ±0  0 ❌ ±0 

Results for commit f40148b. ± Comparison against base commit e77332e.

@mcollovati mcollovati added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit 90c3174 May 8, 2026
31 checks passed
@mcollovati mcollovati deleted the geo-avail-signal branch May 8, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants