search

srounet / Pymem

A python library for windows, providing the needed functions to start working on your own with memory editing.
MIT License
303 stars 45 forks source link

Use poetry for dependency management #92

Closed StarrFox closed 6 months ago

StarrFox commented 1 year ago

closes https://github.com/srounet/Pymem/issues/91

This pull request switches to a pyproject.toml file which uses poetry, it also attempts to adapt the appveyor to use it

It also removes 3.6 from tests as it is EOL and adds 3.9 and 3.10

StarrFox commented 1 year ago

the test failure seems to be because it thinks python is 2.7? I'm not sure why this happens

re: https://ci.appveyor.com/project/srounet/pymem/builds/45061493/job/huhusjsgefgxbpan

python --version Python 2.7.17

srounet commented 1 year ago

I need to learn more about poetry and see how it would benefits to pymem

StarrFox commented 7 months ago

seems I'm still getting trolled by appveyor https://ci.appveyor.com/project/srounet/pymem/builds/48612762/job/jcb4t99y8wn3pabe when I merged the branches it got the old content back

StarrFox commented 7 months ago

finally working with appveyor

srounet commented 7 months ago

I'm still not familiar with poetry, but I've heard good things about it.

As long as it des not break the automatic release process, I see no reasons why not to merge it.

StarrFox commented 7 months ago

think this is ready to merge now, anyone see any other issues?

StarrFox commented 7 months ago

this check failure just seems like appveyor trolling again, the same things passed on the "branch" version

I think it just timed out or something

monkeyman192 commented 7 months ago

Is it possible to just restart the job? Might need srounet to do it if you don't have permission, or just push some other trivial change to force it to re-run...

StarrFox commented 7 months ago

I think it's fine to ignore since they passed for the branch set, @srounet ok to merge this?

monkeyman192 commented 7 months ago

I think it's fine to ignore since they passed for the branch set, @srounet ok to merge this?

This will be something that should be easier to manage once it's moved to github actions I think...

StarrFox commented 7 months ago

I think it's fine to ignore since they passed for the branch set, @srounet ok to merge this?

This will be something that should be easier to manage once it's moved to github actions I think...

hoping to start working on that after this is merged

StarrFox commented 6 months ago

looks ready to merge to me