Redac/redac-rouille
3
0
Fork 0
Code Issues 22 Pull requests Projects Releases Packages Wiki Activity

fix/redac-aled-52: fix issue #52 and #34 in redac aled #53

Merged
jthulhu merged 2 commits from fix/redac-aled-52 into main 2025-10-29 11:38:15 +01:00
Conversation 4 Commits 2 Files changed 2 +6 -2
mboyer02 commented 2025-10-22 10:05:48 +02:00
Owner
Copy link
No description provided.
jthulhu was assigned by mboyer02 2025-10-22 10:06:04 +02:00
mboyer02 commented 2025-10-22 10:06:50 +02:00
Author
Owner
Copy link

@jthulhu tu peux check que mon changement de print! à println! dans main.rs n'est pas un problème ?

@jthulhu tu peux check que mon changement de `print!` à `println!` dans `main.rs` n'est pas un problème ?
jthulhu approved these changes 2025-10-22 11:01:08 +02:00
src/main.rs Outdated
@ -44,0 +43,4 @@
Action::Aled => println!(
"{}\n BOcadoc: {} ",
Cli::command().render_help().ansi(),
r"https://bocal.cof.ens.fr/www/bocadoc/"
jthulhu commented 2025-10-22 11:00:56 +02:00
Owner
Copy link

Pourquoi mettre cette url à part, plutôt que de simplement println!("{}\n Bocadoc: https://...", ...)?

Pourquoi mettre cette url à part, plutôt que de simplement `println!("{}\n Bocadoc: https://...", ...)`?
mboyer02 commented 2025-10-22 11:03:37 +02:00
Author
Owner
Copy link

parce que justement je comprenais pas pourquoi il y avait un % à la fin du code (je connaissais pas la diff entre print! et println!)

parce que justement je comprenais pas pourquoi il y avait un `%` à la fin du code (je connaissais pas la diff entre print! et println!)
jthulhu commented 2025-10-22 11:05:21 +02:00
Owner
Copy link

println!(...) c'est comme print!(...) sauf que ça rajoute un '\n' à la fin.

`println!(...)` c'est comme `print!(...)` sauf que ça rajoute un `'\n'` à la fin.
jthulhu commented 2025-10-22 11:02:05 +02:00
Owner
Copy link

@mboyer02 wrote in #53 (comment):

@jthulhu tu peux check que mon changement de print! à println! dans main.rs n'est pas un problème ?

Oui c'est ok, normalement Cli::command().render_help().ansi() met déjà un retour à la ligne à la fin, donc là tu mets une ligne vide entre l'aide et le BOcadoc: .... Jsp si c'est voulu.

@mboyer02 wrote in https://git.dgnum.eu/Redac/redac-rouille/pulls/53#issuecomment-18867: > @jthulhu tu peux check que mon changement de `print!` à `println!` dans `main.rs` n'est pas un problème ? Oui c'est ok, normalement `Cli::command().render_help().ansi()` met déjà un retour à la ligne à la fin, donc là tu mets une ligne vide entre l'aide et le ` BOcadoc: ...`. Jsp si c'est voulu.
mboyer02 commented 2025-10-22 11:04:16 +02:00
Author
Owner
Copy link

Oui je veux une ligne vide, ça c'est voulu

Oui je veux une ligne vide, ça c'est voulu
👍 1
jthulhu force-pushed fix/redac-aled-52 from 446dd7e2d0 to ee9c6da51f 2025-10-29 11:36:14 +01:00 Compare
jthulhu deleted branch fix/redac-aled-52 2025-10-29 11:38:21 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
jthulhu
Labels
Clear labels   
kind
bug

  
kind
easter-egg

  
kind
feature-request

  
kind
not-a-bug

No labels
kind
bug
kind
easter-egg
kind
feature-request
kind
not-a-bug
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
jthulhu
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: Redac/redac-rouille#53
No description provided.
Delete branch "fix/redac-aled-52"

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?