Skip to content

Working PR for ADR-11: Access secrets for AdaptiveRemote from AWS - #131

Draft
jodavis wants to merge 2 commits into
mainfrom
feature/ADR-11-Access-Secrets-from-AWS
Draft

Working PR for ADR-11: Access secrets for AdaptiveRemote from AWS#131
jodavis wants to merge 2 commits into
mainfrom
feature/ADR-11-Access-Secrets-from-AWS

Conversation

@jodavis

@jodavisjodavis commented Mar 25, 2026

Copy link
Copy Markdown
Owner

@github-actions

github-actionsBot commented Mar 25, 2026

Copy link
Copy Markdown

Test Results

404 tests +3 404 ✅ +3 2m 22s ⏱️ +13s
10 suites ±0 0 💤 ±0 
10 files ±0 0 ❌ ±0 

Results for commit c0350d6. ± Comparison against base commit c4ca900.

This pull request removes 3 and adds 5 tests. Note that renamed tests count towards both.
,False)
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesKeyNameAsync (Hello
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesValueAsync (Invalid
AdaptiveRemote.Services.Lifecycle.AcceleratedServicesTests ‑ AcceleratedServices_StartupConfig_CommandLineArgsOverrideEnvironmentVariables
AdaptiveRemote.Services.Lifecycle.AcceleratedServicesTests ‑ AcceleratedServices_StartupConfig_ReadsCommandLineArgs
AdaptiveRemote.Services.Lifecycle.AcceleratedServicesTests ‑ AcceleratedServices_StartupConfig_ReadsEnvironmentVariables
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesKeyNameAsync (Hello
,False)
AdaptiveRemote.Services.ProgrammaticSettings.PersistSettingsTests ‑ PersistSettings_Set_ValidatesValueAsync (Invalid
,False)

♻️ This comment has been updated with latest results.

@jodavis
jodavisforce-pushed the feature/ADR-11-Access-Secrets-from-AWS branch from e504373 to 55dabc5CompareMarch 25, 2026 18:51
@jodavisjodavis changed the title Spec for ADR-11: Access secrets for AdaptiveRemote from AWSWorking PR for ADR-11: Access secrets for AdaptiveRemote from AWSMar 25, 2026
Sign up for freeto 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

@jodavis@ElwoodMoves