openium/resgen.swift
1
0
Fork 0
Code Pull Requests Releases 14 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
Member
Copy Link
Copy Source
No description provided.
q.bandera added 3 commits 2024-04-11 11:19:18 +02:00
Correction des commentaires des strings
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
0bd6c3c2d4
Update JenkinsFile Xcode version
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
d21ad9d1ea
Modification de l'affichage des commentaires
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
Copy Source

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
Affichage du commentaire même si nil ou empty 5d4e461933
q.bandera commented 2024-04-11 14:24:30 +02:00
Author
Member
Copy Link
Copy Source

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
Copy Source

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
Changed the condition for moe visibility 1d7fc76340
q.bandera added 1 commit 2024-04-12 16:09:56 +02:00
Première implémentation des xcstrings 0d651b810f
q.bandera added 1 commit 2024-04-12 16:25:12 +02:00
Correction du test testGenerateXcStringsRootObject 6c3f3a8982
q.bandera added 1 commit 2024-04-12 16:45:12 +02:00
Fix equatable properties for arrays ba07005b13
q.bandera added 1 commit 2024-04-12 17:15:17 +02:00
Gestion des commentaires 209ba49e3f
q.bandera added 1 commit 2024-04-15 16:51:21 +02:00
Correction de l'option pour le fichier xcstrings 27f86f5c4d
q.bandera added 1 commit 2024-04-17 09:44:13 +02:00
DEVTOOLS-181 Gérer le tag noTranslation pour les xcstrings fb2ddb2227
q.bandera marked the pull request as ready for review 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
Copy Source

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
Clear labels
android
Concerne l'app android
 backend
Concerne le back-end
 devops
Concerne l'infrastructure ou la CI/CD
 frontweb
Concerne le front-end
 iOS
Concerne l'app iOS
 kmp
Concerne KMP
 resgen
Concerne Resgen
maintenance
dependencies
Mise à jour ou gestion des dépendances
maintenance
refactor
Refactoring sans changement fonctionnel
priority
high
Priorité élevée
priority
low
Priorité faible
priority
medium
Priorité moyenne
security
audit
Élément nécessitant une revue de sécurité
security
vulnerability
Faille de sécurité ou risque associé
status
blocked
Bloqué par une dépendance
status
ready
Prêt pour revue
status
wip
En cours de travail
type
bug
Correction de bug
type
chore
Tâche de maintenance ou technique
type
docs
Documentation
type
feature
Nouvelle fonctionnalité
type
improvement
Amélioration ou optimisation
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
a.chabozy (Audrey Chabozy) a.valin (Arthur Valin) a.vallet (Aurélien Vallet) b.lambert (Benjamin Lambert) c.guelle (Clémentine Guelle) c.samson (Clément Samson) e.berlinet (Emilie Berlinet) e.guilbault (Emma Guilbault) g.fernandes (Guillaume Fernandes) h.peytavin (Hugues Peytavin) j.neboit (Jean-Charles Neboit) jenkins (Doc Hudson) k.renella (Kévin Renella) l.adlington (Laetitia Adlington) l.godart (Ludovic Godart) l.perret (Loris Perret) l.riviere (Laura Rivière) l.seguin (Léane Seguin) l.zborowski (Lucas Zborowski) n.chaduc (Nicolas Chaduc) o.coudert (Ophélie COUDERT) o.goutet (Olivier Goutet) p.hurdebourcq (Pierre Hurdebourcq) q.bandera (Quentin Bandera) r.bergoin (Richard Bergoin) r.bossin (Rodolphe Bossin) redmansible (Hans Heeble) s.mellouk (Solène Mellouk) s.tabaka (Simon Tabaka) t.leduc (Thomas Leduc) t.schmitt (Thibaut Schmitt) watea-azure-pipeline (Access Watea Azure Pipeline to SwiftiumKit)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openium/resgen.swift#10