Skip to content

fix(liveness): Remove Amplify.API dependency from Liveness - #327

Merged
mattcreaser merged 1 commit into
mainfrom
mattcreaser/remove-api-dependency
Jul 24, 2026
Merged

mattcreaser merged 1 commit into
mainfrom
mattcreaser/remove-api-dependency

Conversation

@mattcreaser

Copy link
Copy Markdown
Member

Issue #, if available:
Liveness depended on Amplify API despite not making any API calls. I had to do some digging but eventually found that this dependency was originally added accidentally in a migration in 2023.

Description of changes:

  • Removed the API dependency

How did you test these changes?

  • Liveness session works as normal

Documentation update required?

  • No
  • Yes (Please include a PR link for the documentation update)

General Checklist

  • Added Unit Tests
  • Added Integration Tests
  • Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
  • Ensure commit message has the appropriate scope (e.g fix(liveness): message, fix(authenticator): message, fix(all): message)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mattcreaser
mattcreaser requested review from a team as code owners July 23, 2026 18:45
@mattcreaser
mattcreaser enabled auto-merge (squash) July 23, 2026 18:47
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.39%. Comparing base (a5c66cb) to head (41451ab).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #327   +/-   ##
=======================================
  Coverage   37.39%   37.39%           
=======================================
  Files         123      123           
  Lines        5019     5019           
  Branches      707      707           
=======================================
  Hits         1877     1877           
  Misses       2948     2948           
  Partials      194      194           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mattcreaser
mattcreaser merged commit 487ae08 into main Jul 24, 2026
7 checks passed
@mattcreaser
mattcreaser deleted the mattcreaser/remove-api-dependency branch July 24, 2026 15:06
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