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

Adding Shiny attribute to Pokemon #5979

Merged
merged 4 commits into from
Mar 28, 2017
Merged

Conversation

davidakachaos
Copy link
Contributor

@davidakachaos davidakachaos commented Mar 27, 2017

See #5975

Adding attribute "Shiny" to pokemons.

@mention-bot
Copy link

@davidakachaos, thanks for your PR! By analyzing the history of the files in this pull request, we identified @marksweb, @Gobberwart and @Napator to be potential reviewers.

@pogarek
Copy link
Contributor

pogarek commented Mar 27, 2017

Thanks for the PR
You would need to fix tests as well...
You can run tests at any time:
` python -m unittest discover -v -p "*_test.py"

if you need help with that , step by to discord. :)

@pogarek
Copy link
Contributor

pogarek commented Mar 27, 2017

by fixing test I mean, modify code of tests so it matches actual results that tests results right now

@davidakachaos
Copy link
Contributor Author

davidakachaos commented Mar 27, 2017 via email

@davidakachaos
Copy link
Contributor Author

@pogarek fixed the tests 😄

@pogarek
Copy link
Contributor

pogarek commented Mar 28, 2017

👍

Approved with PullApprove

@pogarek pogarek merged commit 16f4e2b into PokemonGoF:dev Mar 28, 2017
@pogarek
Copy link
Contributor

pogarek commented Mar 28, 2017

merged #5979

@davidakachaos davidakachaos deleted the shiny_pokemon branch August 23, 2017 07:55
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.

3 participants