Skip to content

Updated dependency requirements#23

Merged
jecarr merged 3 commits intomainfrom
lxml-update
Jan 28, 2025
Merged

Updated dependency requirements#23
jecarr merged 3 commits intomainfrom
lxml-update

Conversation

@jecarr
Copy link
Member

@jecarr jecarr commented Dec 12, 2024 •
edited
Loading

  • lxml compatibility update
  • Search engine settings update based on https://discuss.freedombox.org/t/searx-is-awesome-but-the-default-settings-not-so-much/2648 (recommended by @KadeMorton)
    • Google disabled - had issues in dev-environment with exceeding search-limits
    • Brave disabled - frequently had content-encoding: br, failed to decode errors
    • Ygg - is disabled but still had errors initialising in the log - possibly my misunderstanding of how the search engines are initialised
    • SoundCloud disabled - thought this would be relevant for music searches only, please change if using Tracery locally

@jecarr
Updated dependency requirements
882cff6 
- Brotli/certifi minor-version upgrade
- lxml major-version upgrade
- Removed reference to lxml.etree._ElementStringResult following lxml/lxml@eba7934
@jecarr
Added Python 3.13 to CI tests
67a2b2f
@jecarr
Copy link
Member Author

jecarr commented Dec 12, 2024

Test failure due to pylint-dev/pylint#10112

KadeMorton
KadeMorton previously approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@KadeMorton KadeMorton left a comment

Choose a reason for hiding this comment

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

Discussed and approved.

@jecarr jecarr merged commit cd567d9 into main Jan 28, 2025
8 checks passed
@jecarr jecarr deleted the lxml-update branch January 28, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KadeMorton KadeMorton KadeMorton left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jecarr @KadeMorton