Skip to content

Fix Pydantic V2 deprecation warnings and requirements compatibility#5

Open
Copilot wants to merge 3 commits intomainfrom
copilot/fix-issue-with-login
Open

Fix Pydantic V2 deprecation warnings and requirements compatibility#5
Copilot wants to merge 3 commits intomainfrom
copilot/fix-issue-with-login

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 3, 2026

Fixed critical errors in the repository that were preventing proper functionality with Python 3.12 and modern Pydantic versions.

Status

Successfully fixed all errors and warnings in the codebase:

  • ✅ Fixed Pydantic V2 deprecation warnings
  • ✅ Updated requirements.txt for Python 3.12 compatibility
  • ✅ All 15 tests passing
  • ✅ No security vulnerabilities found

Changes Made

1. Pydantic V2 Compatibility (dlplus/config/settings.py)

  • Updated to use modern ConfigDict instead of deprecated Config class
  • Removed deprecated env parameter from all Field definitions
  • Environment variables are now properly read using Pydantic V2 conventions

2. Requirements Compatibility (requirements.txt)

  • Fixed torch version incompatibility (was 2.1.1, now >=2.2.0,<3.0.0 for Python 3.12 support)
  • Changed from strict version pinning (==) to compatible release specifiers (~=)
  • This prevents breaking changes from major version updates while allowing safe patches and minor updates

Testing

  • ✅ All 15 tests passing
  • ✅ Server starts without errors
  • ✅ No deprecation warnings
  • ✅ Code review passed with no issues
  • ✅ CodeQL security scan found 0 vulnerabilities

The repository is now fully functional with Python 3.12 and modern dependencies.

Original prompt

اهلاً


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix issue with user login functionality No changes - problem statement contains no requirements Feb 3, 2026
@wasalstor-web
Copy link
Copy Markdown
Owner

@copilot اصلح الاخطاء واعتمد

@wasalstor-web wasalstor-web marked this pull request as ready for review February 3, 2026 15:45
Copilot AI requested a review from wasalstor-web February 3, 2026 15:45
Copilot AI review requested due to automatic review settings February 3, 2026 15:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: wasalstor-web <230709381+wasalstor-web@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 3, 2026

@copilot اصلح الاخطاء واعتمد

تم إصلاح الأخطاء بنجاح ✅

الأخطاء التي تم إصلاحها:

  1. تحديثات Pydantic V2 - إزالة التحذيرات المهجورة
  2. تحديث requirements.txt - إصلاح تعارضات الإصدارات
  3. جميع الاختبارات تعمل بنجاح (15/15) ✅

Commit: 7bd6226

Co-authored-by: wasalstor-web <230709381+wasalstor-web@users.noreply.github.com>
Copilot AI changed the title No changes - problem statement contains no requirements Fix Pydantic V2 deprecation warnings and requirements compatibility Feb 3, 2026
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