Skip to content

Feat: Add python stub#53

Closed
RF-Tar-Railt wants to merge 5 commits into
amitdev:masterfrom
RF-Tar-Railt:feat/stub
Closed

Feat: Add python stub#53
RF-Tar-Railt wants to merge 5 commits into
amitdev:masterfrom
RF-Tar-Railt:feat/stub

Conversation

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor

#49

@AndreMiras

Copy link
Copy Markdown
Contributor

Hi @RF-Tar-Railt, would you mind rebasing this PR to master? Recent changes made it possible to run tests as part of the PR to prevent regressions, see https://github.com/amitdev/lru-dict/pull/54/files#r1359945835

@miili

miili commented Nov 2, 2023

Copy link
Copy Markdown
Contributor

Where did the stubs go? Why were they reverted. IMO those are important in modern Python development.

amitdev and others added 4 commits November 3, 2023 11:56
Note that we (cross) compile for both arm64 and the universal2.
This may feel redundant, it can still be interesting to have both.
While universal2 is more convenient arm64 is lighter.

Also drop Python 3.7 (end-of-life 2023-06-27).
Add Python 3.12 and pypy-3.10 support.

Bump CI actions versions as well.
@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

Hi @RF-Tar-Railt, would you mind rebasing this PR to master? Recent changes made it possible to run tests as part of the PR to prevent regressions, see #54 (files)

updated.

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

@amitdev
All Build and Tests have passed.
image

Please note that I did not modify the version

@amitdev

amitdev commented Nov 3, 2023

Copy link
Copy Markdown
Owner

It seems to have some conflicts @RF-Tar-Railt

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

It seems to have some conflicts @RF-Tar-Railt

But I don't seem to have the authority to resolve conflicts

amitdev added a commit that referenced this pull request Nov 5, 2023
@amitdev amitdev mentioned this pull request Nov 5, 2023
@amitdev

amitdev commented Nov 5, 2023

Copy link
Copy Markdown
Owner

Sorry, can you raise another PR? When I try to resolve conflict I get #56 which doesn't seem to include all changes?

@AndreMiras

AndreMiras commented Nov 5, 2023

Copy link
Copy Markdown
Contributor

All we need is probably:

git fetch --all
git reset --hard origin/master
git cherry-pick bf2faab891e0dae43108f14d875859ed995bf084
git push --force

This is assuming origin is git@github.com:amitdev/lru-dict.git, but depending on your setup/fork replace origin by upstream

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

All we need is probably:

git fetch --all
git reset --hard origin/master
git cherry-pick bf2faab891e0dae43108f14d875859ed995bf084
git push --force

This is assuming origin is git@github.com:amitdev/lru-dict.git, but depending on your setup/fork replace origin by upstream

use in my fork or ?

@amitdev amitdev mentioned this pull request Nov 5, 2023
@AndreMiras

AndreMiras commented Nov 5, 2023

Copy link
Copy Markdown
Contributor

Yes you can totally do it from your fork, say upstream is configured with git@github.com:amitdev/lru-dict.git and your fork is origin with git@github.com:RF-Tar-Railt/lru-dict.git you can:

git fetch --all
git reset --hard upstream/master # erase with master change
git cherry-pick bf2faab891e0dae43108f14d875859ed995bf084
git push --force

I can walk you through on Discord I'm AndreMiras#3439

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

done.

@AndreMiras

Copy link
Copy Markdown
Contributor

Ahah I'm not sure why the PR closed though 😅

@RF-Tar-Railt

Copy link
Copy Markdown
Contributor Author

Ahah I'm not sure why the PR closed though 😅

#58 merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants