Skip to content

Remove Planet routes#255

Merged
dmannarino merged 2 commits intostagingfrom
remove_planet_routes
Jan 6, 2026
Merged

Remove Planet routes#255
dmannarino merged 2 commits intostagingfrom
remove_planet_routes

Conversation

@dmannarino
Copy link
Member

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
  • Make sure you are making a pull request against the develop branch (left side). Also you should start your branch off our develop.
  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

@dmannarino dmannarino changed the base branch from production to staging January 6, 2026 18:43
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.95%. Comparing base (41bfa4f) to head (cfbaf15).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           staging     #255      +/-   ##
===========================================
- Coverage    71.36%   70.95%   -0.41%     
===========================================
  Files           79       78       -1     
  Lines         2762     2734      -28     
===========================================
- Hits          1971     1940      -31     
- Misses         791      794       +3     
Flag Coverage Δ
unittests 70.95% <ø> (-0.41%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmannarino dmannarino merged commit a003694 into staging Jan 6, 2026
5 checks passed
@dmannarino dmannarino deleted the remove_planet_routes branch January 6, 2026 21:26
dmannarino added a commit that referenced this pull request Jan 7, 2026
…in, GH is confused) (#256)

* Remove Planet routes (#255)
* Get lambda layer and core infra values from SSM (Parameter Store) (#236)

---------

Co-authored-by: Dan Scales <dan.scales@gmail.com>
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.

3 participants