-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
Allow merging patches without commit. #3
Conversation
@@ -136,7 +136,7 @@ def _move_modules(self, temp_repo, destination): | |||
|
|||
class Patch(object): | |||
|
|||
def __init__(self, url, branch, commit): | |||
def __init__(self, url, branch, commit=None): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey,
Quel est le scenario que tu cherches a couvrir avec le commit optionel?
J'aime bien le fait de forcer le commit pour pouvoir facilement reproduire et ca stabilise les builds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est vrai, je ne suis pas certain également.
C'est dans le but de pointer sur un pull request d'un autre intégrateur. L'avantage, c'est que s'il y a un rebase, ça ne pette pas l'intallation.
Mais tu as raison. On devrait toujours freezer les patches.
commit=self.commit_rev, | ||
patches=[ | ||
{ | ||
"url": "https://github.com/savoirfairelinux/l10n-canada", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mineur, je ne ferai pas reference a un repo sfl. Est ce qu'on pourrais avoir la meme chose avec un repo OCA?
sfl n'est pas assez stable pour être une réference dans un test :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c'est vrai ;)
@foutoucour merci de ton avis, je vais fermer ce PR. Pas convaincu moi-même que ça vaut la peine. Concernant le repo SFL, on arrangera ça plus tard. |
On peut se faire un script qui permet de rappatrier un répo vers numigi pour ce genre de cas. |
No description provided.