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

Corrige une erreur bloquante dans le code de l'impôt #1031

Merged
merged 1 commit into from
Jul 6, 2018

Conversation

claireleroy
Copy link
Contributor

Erreur de noms de variable issue de la PR #1018 qui empêche le code de l'impôt de tourner

@claireleroy claireleroy requested a review from benjello July 6, 2018 09:50
@claireleroy claireleroy merged commit 656b8ef into master Jul 6, 2018
@benjello
Copy link
Member

benjello commented Jul 6, 2018

Bizarre que les tests soient passer sans qu'on te demande de changer le CHANGELOG ?
@Anna-Livia @sandcha @maukoquiroga est-ce normal ?

@benjello
Copy link
Member

benjello commented Jul 6, 2018

Bon je viens de recevoir le mail de CircleCI qui me dis qu'il ne peut pas déployer le paquet car pas de changement de version. Mais pourquoi ne vérifie-t-on plus que le bump ait lieu pour accepter la PR ?

@bonjourmauko
Copy link
Member

bonjourmauko commented Jul 6, 2018

Hi @benjello, effectivement je crois qu'il s'agit d'un bug.

capture d ecran 2018-07-06 a 12 42 34

L'exit code devrait être n > 0, et pas 0. cc @fpagnoux

if ! $(dirname "$BASH_SOURCE")/detect-functional-changes.sh
then
    echo "No need for a version update."
    exit 0
fi

capture d ecran 2018-07-06 a 12 52 36

@bonjourmauko
Copy link
Member

J'ai commencé #1032, est-ce que tu peux le prendre pour updater la version et le changelog @benjello ?

@Anna-Livia Anna-Livia deleted the corrige-erreur-impot branch July 6, 2018 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants