search

veekun / pokedex

more than you ever wanted to know about Pokémon
MIT License
1.44k stars 637 forks source link

Meltan and Melmetal adding #292

Closed ks129 closed 3 years ago

ks129 commented 4 years ago

Added information about Meltan and Melmetal + Double Iron Bash move.

Naramsim commented 4 years ago

@eevee @magical, what do you think about this contribution? Is it complete and can be merged?

magical commented 4 years ago

What's the source of this data? It looks like it was added by hand, which we usually try to avoid doing.

Also, if we're adding data from LGP&E I'd rather do it all at once. It seems weird to just have these two pokemon.

Naramsim commented 4 years ago

Yep, I think that @ks129 did everything by hand. Can you suggest some ways to extract this kind of data programmatically? Which resources does Veekun usually use?