pennsignals / dsdk

Data Science Deploy Kit
MIT License
8 stars 7 forks source link

Replace pkg_resources with importlib #88

Closed jlubken closed 3 months ago

jlubken commented 10 months ago

Deprecation warning in projects using dsdk indicate that use of pkg_resources needs replaced with importlib.metadata. Importlib is also faster.

SUBINMa commented 10 months ago

Oops

On Tue, Feb 6, 2024, 2:12 PM Graydon @.***> wrote:

@.**** approved this pull request.

LGTM, just updating checkout version and packages.

— Reply to this email directly, view it on GitHub https://github.com/pennsignals/dsdk/pull/88#pullrequestreview-1866124183, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABL6SJPGBTNXQGYUU245UKDYSJ6ADAVCNFSM6AAAAABC4PFJZKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNRWGEZDIMJYGM . You are receiving this because you are subscribed to this thread.Message ID: @.***>

SUBINMa commented 9 months ago

Hi just FYI , I am still getting these emails .

On Tue, Feb 6, 2024 at 2:12 PM Graydon @.***> wrote:

@.**** approved this pull request.

LGTM, just updating checkout version and packages.

— Reply to this email directly, view it on GitHub https://github.com/pennsignals/dsdk/pull/88#pullrequestreview-1866124183, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABL6SJPGBTNXQGYUU245UKDYSJ6ADAVCNFSM6AAAAABC4PFJZKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNRWGEZDIMJYGM . You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Subin

jlubken commented 3 months ago

Updated with addition of 3.12 elsewhere.