joxeankoret / diaphora

Diaphora, the most advanced Free and Open Source program diffing tool.
http://diaphora.re
GNU Affero General Public License v3.0
3.58k stars 371 forks source link

Update to Pygments 2.13.0 #240

Closed CookiePLMonster closed 1 year ago

CookiePLMonster commented 1 year ago

This fixes an incompatibility with Python 3.11 that upgraded this 6-year-old deprecation warning to an error: https://bugs.python.org/issue47066

It's an easy fix HERE, but at this rate it might be a better idea to update to the latest Pygments and get this fix "automatically".

I have verified the following still work, and syntax highlighting works as expected in those cases:

  1. Creating a new diff via diaphora.py between two already generated databases.
  2. Loading an existing diff file via diaphora-load.py.

I just dropped the entire Pygments 2.13.0 into the repo, I am not aware of any omitted files and/or custom changes in your current copy in the repository.

lgtm-com[bot] commented 1 year ago

This pull request introduces 167 alerts and fixes 12 when merging 925d6ab27da7c2e56064c4076daa9610e6e4bd6d into be78f2ce77711c607aa14c6c6c67408f44864bf1 - view on LGTM.com

new alerts:

fixed alerts:

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

joxeankoret commented 1 year ago

Let me take a look to the whole because I haven't tested at all, yet, Python 3.11. I'm sick with covid right now so, give me a few days.

joxeankoret commented 1 year ago

Done with 9a852b5c04c318f1844821d4a17026204c395155. Thank you very much!