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 missing matches #3

Merged
merged 5 commits into from
Jan 18, 2016
Merged

Conversation

markusz
Copy link
Contributor

@markusz markusz commented Jan 17, 2016

Some matches are missing in the generated JSON file, although being in the source files (see individual commits).

This pull request adds the missing matches directly to the JSONized project, since the source data is correct and the error must have happened during processing and Note: [..] please do NOT edit the JSON files but the source files in the country repos e.g.: is not a feasible solution here

I added kicker.de links to verify the correctness of my changes

markusz pushed a commit to markusz/es-espana that referenced this pull request Jan 17, 2016
After closer investigation, openfootball/football.json#3 seems to be caused by typos in the source data
markusz pushed a commit to markusz/de-deutschland that referenced this pull request Jan 17, 2016
After closer investigation, openfootball/football.json#3 seems to be caused by typos in the source data
markusz pushed a commit to markusz/eng-england that referenced this pull request Jan 17, 2016
After closer investigation, openfootball/football.json#3 seems to be caused by typos in the source data
markusz pushed a commit to markusz/it-italy that referenced this pull request Jan 17, 2016
After closer investigation, openfootball/football.json#3 seems to be caused by typos in the source data
@markusz
Copy link
Contributor Author

markusz commented Jan 17, 2016

After closer investigation, this seems to be caused by typos in the source data after all. I suppose this pull request will be redundant once the others to the source data are merged and files are regenerated

@geraldb
Copy link
Member

geraldb commented Jan 18, 2016

Thanks for your efforts and the fix in the source. Great help. Note: Changes here get overwritten with the next build (but thanks to your fixes the missing matches should be now included). Thanks again.

geraldb added a commit that referenced this pull request Jan 18, 2016
@geraldb geraldb merged commit e07925f into openfootball:master Jan 18, 2016
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

2 participants