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

Ajout de nouveaux identifiers très utiles pour faire un programme #68

Open
wants to merge 5 commits into
base: principale
Choose a base branch
from

Conversation

MartinFillon
Copy link

Ces nouveaux identifieurs, permettent une plus grande utilisations de notre belle langue dans le cadre de la rouille.

Copy link
Owner

@bnjbvr bnjbvr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rends l'arg… pardon, merci pour cette fusio-demande.

Cependant, je vois la mention de mots-clés qui n'existent pas dans la version de Shakespeare, ce qui me pose question sur la manière dont a été générée cette fusio-demande. S'agit-il d'une extraction d'un large modèle de langages, comme FélinTGP ?

rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
"bien_ou" => "ok_or",
"bien_ou_avec" => "ok_or_else",
"en_tant_que_déréférencé" => "as_deref",
"en_tant_que_déréférencé_mutable" => "as_mut_deref",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mais.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne comprends pas votre commentaire. Pouvez vous précisez ?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aucune référence à ce nom n'est trouvable dans la bibliothèque standard Rouille.

"compter" => "count",
"trouve" => "find",
"trouve_en_appliquant" => "find_map",
"tous" => "all",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nous sommes en 2024, pensez printemps les ami·es.

Suggested change
"tous" => "all",
"tous·tes" => "all",

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je propose plutot d'utiliser, tous ou toutes separement pusique cette notation est complexe a utiliser sur un ordinateur

rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
rouille_proc_macro/src/lib.rs Outdated Show resolved Hide resolved
@MartinFillon
Copy link
Author

Bonsoir, j'apprécie votre retour si rapide, je tiens à préciser que toutes ces propositions viennent entièrement de nos idées à moi et mon collègue puisque nous comptons faire un projet scolaire en utilisant la rouille, en effet, quelques erreurs se sont glissées par ici, je tiendrais à les corriger au plus vite.

@MartinFillon MartinFillon requested a review from bnjbvr November 4, 2024 21:27
@MartinFillon
Copy link
Author

J'ai aussi ajouté quelques cas qui je pense peuvent être utilisés.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants