-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Conversation
|
||
class Company extends \Faker\Provider\fr_FR\Company | ||
{ | ||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you talking about line 7 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
@fzaninotto I have amended the commits, thanks |
* @note Randomly took 100 from this list | ||
*/ | ||
protected static $jobTitleFormat = array( | ||
'Elagueur-botteur', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please sort these alphabetically
Usefull when nothing is defined, it relies on fr_FR, since it's the same spoken lang.
|
||
/** | ||
* @link http://www.pole-emploi.fr/candidat/le-code-rome-et-les-fiches-metiers-@/article.jspz?id=60702 | ||
* @note Randomly took 300 from this list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fzaninotto At that point... Why not make it 300 ?
Thanks! |
No description provided.