notify-py icon indicating copy to clipboard operation
notify-py copied to clipboard

update loguru to 0.6.0

Open jnoortheen opened this issue 3 years ago • 1 comments

It has been some time since loguru's new release. There is not any breaking changes. It is better to update the version restriction on https://github.com/ms7m/notify-py/blob/c6f30e0784eb18d3b4424ede706719a5fd7cf7b9/pyproject.toml#L16

jnoortheen avatar Feb 08 '22 10:02 jnoortheen

This will be fixed in the next release.

ms7m avatar May 22 '22 19:05 ms7m

It's been almost a year since this issue was created, and it still hasn't been upgraded. Any updates? @ms7m

Kyngo avatar Jan 24 '23 14:01 Kyngo

@Kyngo this was supposed to be fixed in #47 (~ 2 releases ago).. Pushing out a fix to update the restriant.

ms7m avatar Jan 25 '23 04:01 ms7m

v0.3.41 has been published to PyPi and should loosen up the requirement for loguru. Let me know if you're still experiencing some conflicts.

ms7m avatar Jan 25 '23 18:01 ms7m

After updating notify-py and trying to set loguru version to at least 0.6.0, I get the following message when attempting to install my requirements.txt file:

ERROR: Cannot install loguru>=0.6.0 and notify-py==0.3.41 because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested loguru>=0.6.0
    notify-py 0.3.41 depends on loguru<0.6.0 and >=0.5.3

Kyngo avatar Jan 26 '23 08:01 Kyngo

Give it a try now. (v0.3.42)

~/Development/debugging-notifypy/sample_proj » pip freeze > requirements.txt; cat requirements.txt                                                                                       mustafa@Mustafas-Macbook-Pro
───────┬──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
       │ File: requirements.txt
───────┼──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1   │ loguru==0.6.0
───────┴──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
(venv) ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
~/Development/debugging-notifypy/sample_proj » pip install notify-py                                                                                                                     mustafa@Mustafas-Macbook-Pro
Collecting notify-py
  Using cached notify_py-0.3.42-py3-none-any.whl (649 kB)
Requirement already satisfied: loguru<=0.6.0,>=0.5.3 in ./venv/lib/python3.9/site-packages (from notify-py) (0.6.0)
Installing collected packages: notify-py
Successfully installed notify-py-0.3.42

ms7m avatar Jan 26 '23 10:01 ms7m

Would this restriction be further loosen? Like totally remove the upper limit. Loguru has release its 0.7.2.

shirok1 avatar Apr 25 '24 03:04 shirok1