Drop deprecated pkg_resources

[ajjackson]

This line was causing some deprecation warnings. I think this is the right way to modernise it?

[github-actions]

Test Results

       22 files  ±0         22 suites  ±0   55m 11s ⏱️ + 4m 20s
  1 049 tests ±0    1 043 ✔️ ±0    6 💤 ±0  0 ❌ ±0 
10 430 runs  ±0  10 367 ✔️ ±0  63 💤 ±0  0 ❌ ±0 

Results for commit de87124. ± Comparison against base commit 7042114.

♻️ This comment has been updated with latest results.

[ajjackson]

Oldlibs is failing, presumably there is a (Matplotlib?) version that doesn't like the Path objects and expects a string.

Could convert to string but then it will stay that way forever: tempted to narrow down the problematic version and apply tweak in that case only. Then it is more likely to be cleaned up later...

[ajjackson]

After a bit of binary search, Matplotlib styles support Path objects from version 3.2 (March 2020). It doesn't seem unreasonable to bump the minimum requirement to this.

(Mantid is currently pinned to version 3.6)

[codecov-commenter]

Codecov Report

Merging #287 (de87124) into master (7042114) will not change coverage.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master     #287   +/-   ##
=======================================
  Coverage   95.79%   95.79%           
=======================================
  Files          28       28           
  Lines        3992     3992           
  Branches      803      803           
=======================================
  Hits         3824     3824           
  Misses         99       99           
  Partials       69       69           

Files Changed	Coverage Δ
euphonic/styles/__init__.py	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[ajjackson]

@oerc0122 @mducle Is there any reason to be concerned about bumping the Matplotlib dependency a bit? It's nice to do things with Path objects where possible 😄

[oerc0122]

Just a typo here.

Don't think there should be any concern about bumping.

[mducle]

I don't think so, I think we just use fairly standard Matplotlib methods which haven't changed between version 2 and 3.

[ajjackson]

Great, thanks :-) Will merge when tests pass again.

[oerc0122]

Having taken a look at the docs, the main difference is that you no longer have Python 2 support (which I don't think is an issue at this point) and unless you're mucking around in the backend, the interfaces are much the same for standard calls.