openium/resgen.swift
1
0
Fork 0
Code Pull Requests 1 Releases 7 Activity

xcstrings #10

Merged
q.bandera merged 11 commits from xcstrings into master 2024-04-19 16:59:04 +02:00
Conversation 4 Commits 11 Files Changed 14 +1993 -137
q.bandera commented 2024-04-11 11:19:17 +02:00
Owner
Copy Link
No description provided.
q.bandera added 3 commits 2024-04-11 11:19:18 +02:00
Correction des commentaires des strings
Some checks failed
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
0bd6c3c2d4
Update JenkinsFile Xcode version
Some checks failed
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
d21ad9d1ea
Modification de l'affichage des commentaires
Some checks failed
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
gitea-openium/resgen.swift/pipeline/pr-master There was a failure building this commit
Details
55264d61ad
q.bandera requested review from t.schmitt 2024-04-11 11:19:32 +02:00
t.schmitt requested changes 2024-04-11 12:08:48 +02:00
Dismissed
Sources/ResgenSwift/Strings/Model/Definition.swift Outdated
@ -94,1 +87,3 @@
"""
private func getBaseProperty(lang: String, translation: String, isStatic: Bool, comment: String?) -> String {
if let comment, !comment.isEmpty {
t.schmitt commented 2024-04-11 12:08:36 +02:00
Owner
Copy Link

On va simplifier et éviter la duplcation de la property. Mets toujours le commentaire :

/// Translation in \(lang) :
/// \(translation)
///
/// Comment:
/// \(comment?.isEmpty ? comment : "No comment")

et dans la property tu peux toujours mettre , comment: "\(comment ?? "")"

On va simplifier et éviter la duplcation de la property. Mets toujours le commentaire : ``` /// Translation in \(lang) : /// \(translation) /// /// Comment: /// \(comment?.isEmpty ? comment : "No comment") ``` et dans la property tu peux toujours mettre `, comment: "\(comment ?? "")"`
q.bandera added 1 commit 2024-04-11 14:20:58 +02:00
q.bandera commented 2024-04-11 14:24:30 +02:00
Author
Owner
Copy Link

Obligé de mettre ((comment ?? "").isEmpty ? "No comment" : comment!) car:
avec comment?.isEmpty, j'ai l'erreur Optional type 'Bool?' cannot be used as a boolean
Et je me suis permis de rajouter le comment! pour éviter un comment ?? "" car si il est nul dans le test d'avant on le transforme en "" dans tout les cas

Obligé de mettre \((comment ?? "").isEmpty ? "No comment" : comment!) car: avec comment?.isEmpty, j'ai l'erreur Optional type 'Bool?' cannot be used as a boolean Et je me suis permis de rajouter le comment! pour éviter un comment ?? "" car si il est nul dans le test d'avant on le transforme en "" dans tout les cas
t.schmitt commented 2024-04-11 15:23:34 +02:00
Owner
Copy Link

Obligé de mettre ((comment ?? "").isEmpty ? "No comment" : comment!) car:
avec comment?.isEmpty, j'ai l'erreur Optional type 'Bool?' cannot be used as a boolean
Et je me suis permis de rajouter le comment! pour éviter un comment ?? "" car si il est nul dans le test d'avant on le transforme en "" dans tout les cas

Quand tu veux faire un test de bool avec un optionnel tu auras l'erreur que tu as mis si tu le laisses tel quel. Pour pas avoir l'erreur il suffit d'un comment?.isEmpty == false au runtime ca donne un nil == false ==> false

> Obligé de mettre \((comment ?? "").isEmpty ? "No comment" : comment!) car: > avec comment?.isEmpty, j'ai l'erreur Optional type 'Bool?' cannot be used as a boolean > Et je me suis permis de rajouter le comment! pour éviter un comment ?? "" car si il est nul dans le test d'avant on le transforme en "" dans tout les cas Quand tu veux faire un test de bool avec un optionnel tu auras l'erreur que tu as mis si tu le laisses tel quel. Pour pas avoir l'erreur il suffit d'un `comment?.isEmpty == false` au runtime ca donne un nil == false ==> false
q.bandera added 1 commit 2024-04-11 15:49:11 +02:00
q.bandera added 1 commit 2024-04-12 16:09:56 +02:00
q.bandera added 1 commit 2024-04-12 16:25:12 +02:00
q.bandera added 1 commit 2024-04-12 16:45:12 +02:00
q.bandera added 1 commit 2024-04-12 17:15:17 +02:00
q.bandera added 1 commit 2024-04-15 16:51:21 +02:00
q.bandera added 1 commit 2024-04-17 09:44:13 +02:00
q.bandera changed title from WIP: xcstrings to xcstrings 2024-04-17 14:05:07 +02:00
t.schmitt approved these changes 2024-04-19 16:51:13 +02:00
t.schmitt left a comment
Owner
Copy Link

Ca m'a l'air bien 👍

Ca m'a l'air bien 👍
q.bandera merged commit 4ad15fcded into master 2024-04-19 16:59:04 +02:00
q.bandera deleted branch xcstrings 2024-04-19 16:59:04 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
t.schmitt
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
a.chabozy a.valin a.vallet b.lambert c.guelle c.samson c.villeneuve e.berlinet e.guilbault e.joly g.fernandes h.peytavin j.neboit jenkins k.renella l.adlington l.godart l.legrand l.perret l.riviere l.seguin l.zborowski m.daim n.chaduc o.coudert o.goutet p.hurdebourcq q.bandera r.bergoin r.bossin redmansible s.mellouk s.tabaka t.coulange t.leduc t.schmitt watea-azure-pipeline
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openium/resgen.swift#10
No description provided.
Delete Branch "xcstrings"

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?