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

Added the fake data to generate car brands names #115

Merged
merged 6 commits into from
May 13, 2019

Conversation

felipeflorencio
Copy link
Contributor

Hello I would like to contribute adding the generated data for Car Brands it's update as far I saw, they have tesla so looks really update ;)

Any change please let me know 👍

expect(name).to(equal("BMW"))
}
}

Copy link
Contributor

Choose a reason for hiding this comment

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

@felipeflorencio maybe remove this line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adjusted :) 👍

@dunyakirkali
Copy link
Contributor

dunyakirkali commented May 8, 2019

@felipeflorencio Could you also update the README.md with some usage instructions?

@felipeflorencio
Copy link
Contributor Author

@felipeflorencio Could you also update the README.md with some usage instructions?

Added 👍

@vadymmarkov
Copy link
Owner

Great @felipeflorencio. Could you also add this generator to https://github.com/vadymmarkov/Fakery/blob/master/Sources/Fakery/Faker.swift to make it accessible.

@felipeflorencio
Copy link
Contributor Author

Great @felipeflorencio. Could you also add this generator to https://github.com/vadymmarkov/Fakery/blob/master/Sources/Fakery/Faker.swift to make it accessible.

Done 👍

# Conflicts:
#	Fakery.xcodeproj/project.pbxproj
#	README.md
#	Resources/Locales/en-TEST.json
#	Resources/Locales/en.json
#	Sources/Fakery/Faker.swift
@felipeflorencio
Copy link
Contributor Author

All conflicts are fixed now, a lot of pull request's 💯

@dunyakirkali
Copy link
Contributor

@vadymmarkov ☝️

@vadymmarkov vadymmarkov merged commit 87c2401 into vadymmarkov:master May 13, 2019
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.

None yet

3 participants