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

spelling: milimetre in C08.csv #61

Closed
manfredsc opened this issue Jul 29, 2021 · 3 comments · Fixed by #76
Closed

spelling: milimetre in C08.csv #61

manfredsc opened this issue Jul 29, 2021 · 3 comments · Fixed by #76
Assignees
Milestone

Comments

@manfredsc
Copy link

manfredsc commented Jul 29, 2021

#Branch
https://github.com/wmo-im/CCT/tree/issue61

code 163 in table C08.csv has "Sub-milimetre", which should be "Sub-millimetre", with a double "l".
I guess the spelling "metre" (as opposed to "meter") is OK, as such spelling is used elsewhere, too.

@jitsukoh
Copy link

jitsukoh commented Sep 2, 2021

Thank you @manfredsc for pointing out. We will correct it in the next cycle.
@amilan17 it seems correction needs to be made both in the Manual and the CSV. Could you add labels/projects to this issue?

@jitsukoh jitsukoh added this to the FT2022-1 milestone Sep 2, 2021
@amilan17 amilan17 self-assigned this Sep 2, 2021
amilan17 added a commit that referenced this issue Nov 10, 2021
@amilan17
Copy link
Member

branch is ready

@jitsukoh
Copy link

@amilan17 I confirm that the branch is updated and the issue was moved to the validated status.

@amilan17 amilan17 linked a pull request Nov 11, 2021 that will close this issue
@amilan17 amilan17 closed this as completed Apr 5, 2022
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 a pull request may close this issue.

3 participants