Skip to content
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

[beta 15.9] petite coquille dans le code #3063

Closed
firm1 opened this issue Oct 9, 2015 · 17 comments
Closed

[beta 15.9] petite coquille dans le code #3063

firm1 opened this issue Oct 9, 2015 · 17 comments
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Milestone

Comments

@firm1
Copy link
Contributor

firm1 commented Oct 9, 2015

En zieutant un peut le code, sur cette ligne,on peut lire ceci :

attrs = [
            'pk', 'authors', 'authors', 'subcategory', 'image', 'creation_date', 'pubdate', 'update_date', 'source',
            'sha_draft', 'sha_beta', 'sha_validation', 'sha_public'
        ]

Les plus vigilents auront remarqués qu'il y'a deux fois le "authors" dans la liste. Je suppose que c'est un effet de la fatigue ?

@firm1 firm1 added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Oct 9, 2015
@artragis
Copy link
Member

artragis commented Oct 9, 2015

pour ce genre de code, tu as le droit de faire une PR plutôt que d'ouvrir une issue.

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

@artragis restons sympa, tout le monde n'a pas accès à une machine pour faire des PR, donc il est toujours bon d'avoir un retour même pour des trucs trivials plutôt que de passer à côté de quelque chose.

@pierre-24
Copy link
Member

Non, mais là, il peut passer direct par GH. Soit, la PR est là, et objectivement, y'a rien à tester (sauf si vous voulez TOUT tester :p )

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

Pour passer par GH il faut un fork à jour sur GH...

@artragis
Copy link
Member

pas si sûr. Le système te permet de bien faire les choses (il wrappe autour
de checkout -b .. upstream/release

mais bon pas grave pierre a déjà tout fait.

2015-10-13 11:31 GMT+02:00 Eskimon notifications@github.com:

Pour passer par GH il faut un fork à jour sur GH...


Reply to this email directly or view it on GitHub
#3063 (comment)
.

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

J'ai pas compris :( Mais si tu sais comment garder un fork GH à jour ca m'intéresse vivement :)

@firm1
Copy link
Contributor Author

firm1 commented Oct 13, 2015

Pour passer par GH il faut un fork à jour sur GH...

C'est bien mon problème aussi et je n'ai pas bien compris la manip de @artragis mais si il sait comment faire ça m'interesse aussi.

@pquentin
Copy link
Contributor

En fait on peut éditer directement chaque fichier sur GitHub. Je pense que c'est ce que proposait @artragis. Et ci-dessous, voilà comment reproduire ce que fait GitHub quand on édite un fichier sur un répo sur lequel on ne peut pas commiter.

L'idée est de créer une nouvelle branche locale avec un nom plus ou moins descriptif (patch-1 vs patch-attrs-typo). En supposant que la remote se nomme origin, on peut faire :

git fetch origin  # pour être sûr d'avoir les derniers commits
git checkout -b patch-1 origin/dev  # pour avoir les derniers commits dans la branch patch-1

Et là il est possible de travailler sur patch-1, de pusher sur son fork, et de créer une pull request depuis l'interface GitHub.

@firm1
Copy link
Contributor Author

firm1 commented Oct 13, 2015

Peut-être je n'ai pas précisé mes contraintes, mais je pense qu' @artragis les connais.

  1. j'ai les droits pour commiter sur le depot officiel (mais ce n'est pas ce qu'il faut faire) et du coup la méthode de @pquentin ne marche pas
  2. j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je ne veux pas écraser car il y'a des branches dessus qui m'interesse.

Comment je fais depuis github pour faire les modifs ? J'ai déjà cherché, mais ça n'a pas l'air d'être possible.

Conclusion : j'ai crée l'issue pour que ça ne soit pas perdu :)

@artragis
Copy link
Member

j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je ne
veux pas écraser car il y'a des branches dessus qui m'interesse.

C'est ça que jen e comprends pas. Mon fork n'est pas non plus syncro avec
upstream : ma branche dev est totalement hors de date.

Maintenant la contrainte "1" est la clef du problème : comme tu as les
droits de commits, ils ne te propose pas automatiquement de passer sur ton
fork. Il faut donc qu'à un moment ou à un autre, tu fasses un

git fetch upstream
git checkout -b unebranche upstream/dev
git push origin unebranche

Cela ne détruira pas ton fork.

Le 13 octobre 2015 13:50, firm1 notifications@github.com a écrit :

Peut-être je n'ai pas précisé mes contraintes, mais je pense qu' @artragis
https://github.com/artragis les connais.

  1. j'ai les droits pour commiter sur le depot officiel (mais ce n'est
    pas ce qu'il faut faire) et du coup la méthode de @pquentin
    https://github.com/pquentin ne marche pas
  2. j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je
    ne veux pas écraser car il y'a des branches dessus qui m'interesse.

Comment je fais depuis github pour faire les modifs ? J'ai déjà cherché,
mais ça n'a pas l'air d'être possible.

Conclusion : j'ai crée l'issue pour que ça ne soit pas perdu :)


Reply to this email directly or view it on GitHub
#3063 (comment)
.

@firm1
Copy link
Contributor Author

firm1 commented Oct 13, 2015

Et comment on fait pour taper ces commandes git depuis github ?

Le mar. 13 oct. 2015 13:59, artragis notifications@github.com a écrit :

j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je ne
veux pas écraser car il y'a des branches dessus qui m'interesse.

C'est ça que jen e comprends pas. Mon fork n'est pas non plus syncro avec
upstream : ma branche dev est totalement hors de date.

Maintenant la contrainte "1" est la clef du problème : comme tu as les
droits de commits, ils ne te propose pas automatiquement de passer sur ton
fork. Il faut donc qu'à un moment ou à un autre, tu fasses un

git fetch upstream
git checkout -b unebranche upstream/dev
git push origin unebranche

Cela ne détruira pas ton fork.

Le 13 octobre 2015 13:50, firm1 notifications@github.com a écrit :

Peut-être je n'ai pas précisé mes contraintes, mais je pense qu'
@artragis
https://github.com/artragis les connais.

  1. j'ai les droits pour commiter sur le depot officiel (mais ce n'est
    pas ce qu'il faut faire) et du coup la méthode de @pquentin
    https://github.com/pquentin ne marche pas
  2. j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je
    ne veux pas écraser car il y'a des branches dessus qui m'interesse.

Comment je fais depuis github pour faire les modifs ? J'ai déjà cherché,
mais ça n'a pas l'air d'être possible.

Conclusion : j'ai crée l'issue pour que ça ne soit pas perdu :)


Reply to this email directly or view it on GitHub
<
#3063 (comment)

.


Reply to this email directly or view it on GitHub
#3063 (comment)
.

@artragis
Copy link
Member

pas possible.

C'est là qu'est le problème en fait : comme tu as les droits de commit, y'a
pas automatiquement de création d'une nouvelle branche chez toi (quoi que
c'est peut être possible d'une manière ou d'une autre)

Le 13 octobre 2015 14:39, firm1 notifications@github.com a écrit :

Et comment on fait pour taper ces commandes git depuis github ?

Le mar. 13 oct. 2015 13:59, artragis notifications@github.com a écrit :

j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je ne
veux pas écraser car il y'a des branches dessus qui m'interesse.

C'est ça que jen e comprends pas. Mon fork n'est pas non plus syncro avec
upstream : ma branche dev est totalement hors de date.

Maintenant la contrainte "1" est la clef du problème : comme tu as les
droits de commits, ils ne te propose pas automatiquement de passer sur
ton
fork. Il faut donc qu'à un moment ou à un autre, tu fasses un

git fetch upstream
git checkout -b unebranche upstream/dev
git push origin unebranche

Cela ne détruira pas ton fork.

Le 13 octobre 2015 13:50, firm1 notifications@github.com a écrit :

Peut-être je n'ai pas précisé mes contraintes, mais je pense qu'
@artragis
https://github.com/artragis les connais.

  1. j'ai les droits pour commiter sur le depot officiel (mais ce n'est
    pas ce qu'il faut faire) et du coup la méthode de @pquentin
    https://github.com/pquentin ne marche pas
  2. j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je
    ne veux pas écraser car il y'a des branches dessus qui m'interesse.

Comment je fais depuis github pour faire les modifs ? J'ai déjà
cherché,
mais ça n'a pas l'air d'être possible.

Conclusion : j'ai crée l'issue pour que ça ne soit pas perdu :)


Reply to this email directly or view it on GitHub
<

#3063 (comment)

.


Reply to this email directly or view it on GitHub
<
#3063 (comment)

.


Reply to this email directly or view it on GitHub
#3063 (comment)
.

@firm1
Copy link
Contributor Author

firm1 commented Oct 13, 2015

Donc on revient au problème de départ ... pour ce genre de situation la
meilleure chose que je peux faire c'est une issue.

Le mar. 13 oct. 2015 14:55, artragis notifications@github.com a écrit :

pas possible.

C'est là qu'est le problème en fait : comme tu as les droits de commit, y'a
pas automatiquement de création d'une nouvelle branche chez toi (quoi que
c'est peut être possible d'une manière ou d'une autre)

Le 13 octobre 2015 14:39, firm1 notifications@github.com a écrit :

Et comment on fait pour taper ces commandes git depuis github ?

Le mar. 13 oct. 2015 13:59, artragis notifications@github.com a écrit
:

j'ai un fork qui n'est pas du tout syncro avec l'upstream et que je ne
veux pas écraser car il y'a des branches dessus qui m'interesse.

C'est ça que jen e comprends pas. Mon fork n'est pas non plus syncro
avec
upstream : ma branche dev est totalement hors de date.

Maintenant la contrainte "1" est la clef du problème : comme tu as les
droits de commits, ils ne te propose pas automatiquement de passer sur
ton
fork. Il faut donc qu'à un moment ou à un autre, tu fasses un

git fetch upstream
git checkout -b unebranche upstream/dev
git push origin unebranche

Cela ne détruira pas ton fork.

Le 13 octobre 2015 13:50, firm1 notifications@github.com a écrit :

Peut-être je n'ai pas précisé mes contraintes, mais je pense qu'
@artragis
https://github.com/artragis les connais.

  1. j'ai les droits pour commiter sur le depot officiel (mais ce n'est
    pas ce qu'il faut faire) et du coup la méthode de @pquentin
    https://github.com/pquentin ne marche pas
  2. j'ai un fork qui n'est pas du tout syncro avec l'upstream et que
    je
    ne veux pas écraser car il y'a des branches dessus qui m'interesse.

Comment je fais depuis github pour faire les modifs ? J'ai déjà
cherché,
mais ça n'a pas l'air d'être possible.

Conclusion : j'ai crée l'issue pour que ça ne soit pas perdu :)


Reply to this email directly or view it on GitHub
<

#3063 (comment)

.


Reply to this email directly or view it on GitHub
<

#3063 (comment)

.


Reply to this email directly or view it on GitHub
<
#3063 (comment)

.


Reply to this email directly or view it on GitHub
#3063 (comment)
.

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

hop cadeau pour @Emeric54 qui nous rejoins :)

@pierre-24
Copy link
Member

... S'aurait été avec plaisir, mais je l'ai déjà réglé :p

@GerardPaligot
Copy link
Member

Oh l'autre ! Il fait le travail des nouveaux contributeurs ! :o

@Eskimon Eskimon added this to the Version 15.9 milestone Oct 14, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Oct 14, 2015

Ok

@Eskimon Eskimon closed this as completed Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

6 participants