Skip to content

Push zyxorwwnlzqm - #75

Merged
dunderrrrrr merged 3 commits into
mainfrom
push-zyxorwwnlzqm
Sep 7, 2026
Merged

Push zyxorwwnlzqm#75
dunderrrrrr merged 3 commits into
mainfrom
push-zyxorwwnlzqm

Conversation

@dunderrrrrr

@dunderrrrrr dunderrrrrr commented Sep 7, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes

    • Improved ad parsing compatibility when identifying location information.
  • Chores

    • Updated the minimum supported versions of the application’s dependencies and development tools.
    • Updated the BeautifulSoup version used by the project.
  • Documentation

    • Removed the MkDocs site configuration and stopped excluding the documentation site directory from version control.

DeprecationWarning: The 'text' argument to find()-type methods is deprecated. Use 'string' instead.
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 74a0927e-8325-4e3e-b40e-b28f1f81e888

📥 Commits

Reviewing files that changed from the base of the PR and between e118b3c and 37e046e.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (5)
  • .gitignore
  • blocket_api/ad_parser.py
  • mkdocs.yml
  • pyproject.toml
  • requirements.txt

📝 Walkthrough

Walkthrough

The PR updates BeautifulSoup usage, raises runtime and development dependency minimums, and removes the MkDocs configuration and generated-site ignore rule.

Changes

Parser and project maintenance

Layer / File(s) Summary
BeautifulSoup compatibility
blocket_api/ad_parser.py, pyproject.toml, requirements.txt
McAd.extend uses the string parameter to locate “Plats”. BeautifulSoup minimum versions are raised in both dependency files.
Tooling and documentation cleanup
pyproject.toml, .gitignore, mkdocs.yml
Development dependency minimums are raised. The MkDocs configuration and /site ignore rule are removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch push-zyxorwwnlzqm

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.

@dunderrrrrr
dunderrrrrr merged commit 9b8861c into main Sep 7, 2026
2 of 5 checks passed
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.

1 participant