Skip to content

Commit ec34399

Browse files
CodeWritingCowarminkhoshbin
authored andcommitted
Add sex to names (#474)
* Add name "Jesus" * Fix ISO language code Use "cmn" for Chinese. * Lowercase name * Update 'Michael' Add aliases and translation. Merge "mitch" with "michael" because mitch is an alias for Michael. Delete "mitch.json." * Add gender to name file specs Add gender to file specs in readme.md. Add tests to validate each JSON's gender value. * Update michael.json Resolve merge conflict by using main repo's version of JSON file. * Make sex field optional for now Comment out tests for sex field. They are failing because names in collection don't have that field. Issue #443 * Add tests to check validity of gender Tests run only on name files that contain a gender value. Issue #443 Update README.md to describe sex field as optional. * Update gender tests - Issue #443 Sex field now requires a value to pass test. Also: - Add error messages to explain why a test failed. - Remove test for checking whether a sex value is a string. It's duplicating part of the "should have correct structure" test. * Add sex to 31 names Issue #456 * Add sex to 24 names Issue #456
1 parent 85beb53 commit ec34399

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

55 files changed

+110
-55
lines changed

collection/kevin.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,6 @@
44
"translations": {
55
"por": "kevin",
66
"heb": "קווין"
7-
}
7+
},
8+
"sex": "m"
89
}

collection/kyle.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,6 @@
44
"aliases": [],
55
"translations": {
66
"heb": "קייל"
7-
}
7+
},
8+
"sex": "m"
89
}

collection/lily.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,6 @@
44
"aliases": [],
55
"translations": {
66
"heb": "לילי"
7-
}
7+
},
8+
"sex": "f"
89
}

collection/luigi.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,6 @@
33
"meaning": "Renowned fighter. Form of the French Louis.",
44
"translations": {
55
"heb": "לואיג'י"
6-
}
6+
},
7+
"sex": "m"
78
}

collection/luke.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,6 @@
1313
"gae": "lùc",
1414
"pol": "łukasz",
1515
"heb": "לוק"
16-
}
16+
},
17+
"sex": "m"
1718
}

collection/magnus.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,6 @@
66
"nno": "magnar",
77
"rus": "магнус",
88
"heb": "מגנוס"
9-
}
9+
},
10+
"sex": "m"
1011
}

collection/mahmud.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,6 @@
55
"translations": {
66
"ara": "محمود",
77
"heb": "מחמוד"
8-
}
8+
},
9+
"sex": "m"
910
}

collection/mario.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@
1010
"rus": "марио",
1111
"fas": "ماریو",
1212
"heb": "מריו"
13-
}
13+
},
14+
"sex": "m"
1415
}

collection/mark.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,6 @@
1313
"tha": "มาร์ค",
1414
"mkd": "марк",
1515
"heb": "מארק"
16-
}
16+
},
17+
"sex": "m"
1718
}

collection/martin.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,6 @@
1010
"mkd": "мартин",
1111
"tha": "มาติน",
1212
"heb": "מרטין"
13-
}
13+
},
14+
"sex": "m"
1415
}

0 commit comments

Comments
 (0)