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

Add hydrocannon #10

Merged
merged 2 commits into from
Jul 9, 2018
Merged

Add hydrocannon #10

merged 2 commits into from
Jul 9, 2018

Conversation

celandro
Copy link
Collaborator

@celandro celandro commented Jul 4, 2018

Still don't like us trying to version based off this weird Niantic versioning scheme.

yyyymmdd seems much easier but if there is a place for me to find out what the version number would be today I guess that would be ok too.

@dominicbarnes
Copy link
Contributor

+1 for switching to YYYYMMDD or something else not connected to the in-game version.

Don't Android and iOS version differently? My phone (Android) has 0.107.1, (no updates available) so I'm not sure where 0.108.2 came from. Also, the latest APK from today shows 0.109.2, leading me to be even more confused on how we assign version numbers in this repository.

@BrunnerLivio
Copy link
Contributor

Lets discuss this in #11. This seems to be a bigger issue

@dominicbarnes
Copy link
Contributor

dominicbarnes commented Jul 9, 2018

While the discussion in #11 is ongoing, can we go ahead and get this merged in? (now that the event is over, I'd like Hydro Cannon in my metadata)

@BrunnerLivio BrunnerLivio mentioned this pull request Jul 9, 2018
7 tasks
@BrunnerLivio
Copy link
Contributor

Do not see a problem with that. @dominicbarnes

@BrunnerLivio BrunnerLivio self-requested a review July 9, 2018 16:35
@celandro
Copy link
Collaborator Author

celandro commented Jul 9, 2018 via email

@BrunnerLivio
Copy link
Contributor

@celandro done

@BrunnerLivio BrunnerLivio merged commit d019e38 into master Jul 9, 2018
@Furtif Furtif deleted the 0.110.2 branch July 27, 2019 18:13
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