8000 recipe: don't unnecessarily fetch git remotes. by accumulator · Pull Request #2513 · kivy/python-for-android · GitHub
[go: up one dir, main page]
Skip to content

recipe: don't unnecessarily fetch git remotes. #2513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

recipe: don't unnecessarily fetch git remotes. #2513

wants to merge 3 commits into from

Conversation

accumulator
Copy link
Contributor
@accumulator accumulator 8000 commented Oct 30, 2021

For recipes using a git url, instead of hitting the remote repository on every run, be a bit smarter about it.

Unless self.version in a recipe points to a branch, the remote is only fetched
when self.version is not the current HEAD, and self.version cannot be found in
the local repo.

@accumulator
recipe: don't unnecessarily fetch git remotes.
0e4e97d 
Unless self.version in a recipe points to a branch, the remote is only fetched
when self.version is not the current HEAD, and self.version cannot be found in
the local repo.
@accumulator
recipe: resolve code guidelines issues
45df75a
@accumulator
recipe: disable tty for commands we use the output of
4fb0f22
@misl6 misl6 added need-testing Priority: Low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix labels Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
need-testing Priority: Low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix recipe
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@accumulator @misl6 @kuzeyron
0