Skip to content

Submit current bite#63

Merged
bbelderbos merged 2 commits intoPyBites-Open-Source:mainfrom
rhelmstedter:submit_current_bite
Feb 10, 2025
Merged

Submit current bite#63
bbelderbos merged 2 commits intoPyBites-Open-Source:mainfrom
rhelmstedter:submit_current_bite

Conversation

@rhelmstedter
Copy link
Collaborator

Addresses #62. This is so much nicer. I installed it locally and it's smooth.

As long as eatlocal is aware of the bite (it's in the
~/.eatlocal/.local_bites.json file) it will now submit that bite by
default. Otherwise, it will pull up the iterfzf interface and let the
user choose.
@rhelmstedter
Copy link
Collaborator Author

@bbelderbos just bumping this. I didn't want to merge without you looking at it.

Copy link
Collaborator

@bbelderbos bbelderbos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, looks good and will improve ux a lot, did you update the version in pyproject?

@rhelmstedter
Copy link
Collaborator Author

I have not. I will try to do it this evening.

@bbelderbos
Copy link
Collaborator

bbelderbos commented Jan 29, 2025

thanks, will you commit it here and we merge so I push the new version to PyPI? (let me log an issue to do this automatically when pushing tags ...) -> #37

@bbelderbos
Copy link
Collaborator

Let me do the pypi push on #53, stay tuned ...

@rhelmstedter
Copy link
Collaborator Author

Let me do the pypi push on #53, stay tuned ...

Do you still want me to do the version bump here?

@bbelderbos
Copy link
Collaborator

@rhelmstedter nws I can do that, but can you approve #53 so we try to semi-automate the pushing to pypi? thanks

@bbelderbos
Copy link
Collaborator

thanks @rhelmstedter for reviewing, I merged that one.
I see we have 1.1.5 on pypi, shall we merge the remaining PRs and shoot out a new 1.2 release?

@bbelderbos bbelderbos merged commit 3e744a1 into PyBites-Open-Source:main Feb 10, 2025
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.

2 participants