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

Fix typos found by codespell #43

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DimitriPapadopoulos
Copy link

@DimitriPapadopoulos DimitriPapadopoulos commented Nov 15, 2023

You won't be able to see the changes in the JSON file online, it's a huge 1.5 MB one-liner.

To see the changes, do as follows:

deleted
$ git checkout main
$ python3 -m json.tool android/app/src/main/res/raw/default_list.json  > default_list.json.OLD
$ 
$ git checkout codespell
$ python3 -m json.tool android/app/src/main/res/raw/default_list.json  > default_list.json.NEW
$ 
$ diff default_list.json.OLD default_list.json.NEW 
16610c16610
<             "name": "Faculty of Medicine, Univeristy of Kelaniya",
---
>             "name": "Faculty of Medicine, University of Kelaniya",
45661c45661
<             "name": "Opole University of Technnology",
---
>             "name": "Opole University of Technology",
65541c65541
<             "name": "Technical Univeristy of Lodz",
---
>             "name": "Lodz University of Technology",
71363c71363
<             "name": "Univeristy of Kelaniya",
---
>             "name": "University of Kelaniya",
88603c88603
<             "name": "Wroclaw University of Enviromental and Life Sciences",
---
>             "name": "Wroclaw University of Environmental and Life Sciences",
$ 

Rationale for the changes in the JSON file:

  • University of Kelaniya: it's University, not Univeristy
  • Opole University of Technology: it's Technology, not Technnology
  • Lodz University of Technology: it's University, not Univeristy, plus the name in English is Lodz University of Technology on their web site
  • Wroclaw University of Environmental and Life Sciences: it's Environmental, not Enviromental

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.

1 participant