Skip to content

Apply formatting and fix lint and type errors - #192

Merged
Smoss merged 4 commits into
masterfrom
smoss/format-and-lint-fixes
Oct 14, 2026
Merged

Smoss merged 4 commits into
masterfrom
smoss/format-and-lint-fixes

Conversation

@Smoss

@Smoss Smoss commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

Applies Ruff formatting and import sorting across the project and removes unused imports so the new development checks pass.

Resolves Pyright diagnostics with explicit path and payload types, assertions that successful parsing and metric collection returned values, a corrected manual fixture, and an accurate package export list. Handles Instagram HTTP errors without a response safely and adapts the Mastodon media list to the SDK type. Retains the intentional invalid-tag test with a narrowly scoped diagnostic suppression.

This PR is stacked on smoss/lint-format-typing; merge the tooling PR first. Includes existing formatting changes to the metric collector specification's Python examples.

Validation:

  • make format: completed; subsequent check confirms 44 files formatted.
  • make lint: Ruff passes; Pyright reports 0 errors and 0 warnings.
  • make test: 146 passed, 1 skipped (live API test without credentials).
  • git diff --check: passed.

Comment thread docs/specs/metric-collector.md Outdated
Comment on lines 113 to 116
collected_at: str # ISO8601 UTC, assigned by orchestration
likes: int | None = None # likes/favourites
reposts: int | None = None # reposts/reblogs; None when unavailable
comments: int | None = None # replies/comments

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was intentional, how can we preserve this in the linting rules?

Base automatically changed from smoss/lint-format-typing to master October 14, 2026 00:20
binamkayastha
binamkayastha previously approved these changes Oct 14, 2026

@binamkayastha binamkayastha left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@binamkayastha binamkayastha left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Smoss
Smoss merged commit 857f90f into master Oct 14, 2026
2 checks passed
@Smoss
Smoss deleted the smoss/format-and-lint-fixes branch October 14, 2026 00:33
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