Skip to content

Remove stac_logger dependency - #509

Open
cladbeshreyansh wants to merge 1 commit into
StacDev:mainfrom
cladbeshreyansh:patch-1
Open

cladbeshreyansh wants to merge 1 commit into
StacDev:mainfrom
cladbeshreyansh:patch-1

Conversation

@cladbeshreyansh

@cladbeshreyansh cladbeshreyansh commented Oct 24, 2026 •

Copy link
Copy Markdown

Removed stac_logger dependency from pubspec.yaml because it was never being imported in any file or being used anywhere

Description

Removed the unused stac_logger dependency from packages/stac_core/pubspec.yaml.

  • Verified that stac_logger was not imported or used anywhere in the package.
  • Removed the dependency to keep the package configuration clean and avoid unnecessary dependencies.
  • No application logic or functionality has been changed.

Related Issues

Closes #

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Code refactor
  • Build configuration change
  • Documentation
  • [x ] Chore

Summary by CodeRabbit

  • Chores
    • Updated internal package configuration. This update does not change any visible features or behavior in the app. No user-facing changes are included in this release; existing functionality and workflows remain unchanged. The change is limited to package setup and does not alter how the app works for end users.

Removed stac_logger dependency from pubspec.yaml because it was never being imported in any file or being used anywhere
@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ba647590-fdf5-4511-a46d-7dfdec13d6a4
Commits

Reviewing files that changed from the base of the PR and between acfb49e and eab6c2f.

Files selected for processing: 1
  • packages/stac_core/pubspec.yaml
Files with no reviewable changes: 1
  • packages/stac_core/pubspec.yaml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Walkthrough

The stac_core package removes stac_logger from its declared dependencies.

Changes: Core dependency update

Layer / File(s) Summary
Dependency declaration
packages/stac_core/pubspec.yaml
The dependency list no longer includes stac_logger.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: Minimal · up to eab6c

No actionable merge-blocking risk is identified; this change is mergeable after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing the unused stac_logger dependency.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

This branch has not been deployed

No deployments
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