entrouvert
/
lingo
16
0
Fork 0
Code Issues Pull Requests 6 Releases Wiki Activity

invoicing: add some tests for partial-bookings and multi-check (#82850) #119

Open
lguerin wants to merge 2 commits from wip/82850-invoicing-partial-bookings-multi-check into main
pull from: wip/82850-invoicing-partial-bookings-multi-check
merge into: entrouvert:main
Conversation 1 Commits 2 Files Changed 3 +80 -8
lguerin commented 2023-10-27 16:42:54 +02:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2023-10-27 16:42:54 +02:00
pmarillonnet requested changes 2023-11-14 14:40:08 +01:00
pmarillonnet left a comment
Owner
Copy Link

Pas super fan du format de la PR, à savoir une mini modif dans un gabarit et des tests qui n’ont un peu rien à voir (et en particulier qui n’échouent pas en l’absence de la modif’ de gabarit).

Est-ce que c’est possible d’avoir par exemple deux commits, un avec le one-liner du gabarit et dans les tests un assert judicieux choisi pour tester cette modif’, et l’autre avec les tests déjà présents dans cette version ?

Pas super fan du format de la PR, à savoir une mini modif dans un gabarit et des tests qui n’ont un peu rien à voir (et en particulier qui n’échouent pas en l’absence de la modif’ de gabarit). Est-ce que c’est possible d’avoir par exemple deux commits, un avec le one-liner du gabarit et dans les tests un assert judicieux choisi pour tester cette modif’, et l’autre avec les tests déjà présents dans cette version ?
lguerin commented 2023-11-16 11:34:14 +01:00
Author
Owner
Copy Link

Zut, ça s'est vu :)
Merci pour ta vigilance, je reprends ça

Zut, ça s'est vu :) Merci pour ta vigilance, je reprends ça
lguerin force-pushed wip/82850-invoicing-partial-bookings-multi-check from 5f18cd03ff to 88218fa357 2023-11-16 20:32:07 +01:00 Compare
lguerin requested review from pmarillonnet 2023-11-16 20:32:38 +01:00
All checks were successful
gitea/lingo/pipeline/head This commit looks good
Details
This pull request has changes conflicting with the target branch.
  • lingo/invoicing/templates/lingo/invoicing/invoice.html
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/82850-invoicing-partial-bookings-multi-check main
git pull origin wip/82850-invoicing-partial-bookings-multi-check

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/82850-invoicing-partial-bookings-multi-check
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/lingo#119
No description provided.
Delete Branch "wip/82850-invoicing-partial-bookings-multi-check"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?