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

content update | fix typo #16

Open
marqh opened this issue Apr 21, 2016 · 3 comments
Open

content update | fix typo #16

marqh opened this issue Apr 21, 2016 · 3 comments

Comments

@marqh
Copy link
Contributor

marqh commented Apr 21, 2016

In the
< Contributing to codes.wmo >
section

provide information on how to propose an update fixing a typographical change

@marqh
Copy link
Contributor Author

marqh commented Apr 21, 2016

If you notice a typo in the register, for example, you notice ‘velocity’ is spelled ‘vecolity’, this is how you can get it changed:

  1. The first step is to get the text so you can change it - so download the content in the RDF ttl-plain format.
  2. Open the file with a text editor, find the typo and fix all instances of it.
  3. Go to GitHub (you will need an account, and to be logged in) and go to https://github.com/wmo-registers/code-manuals/tree/master/typo_fixes
  4. Make a new file, using the ‘new file’ button.
  5. Name the file, ending in .ttl
  6. Copy and paste all the amended text into the new file.
  7. When you are proposing the new file, add an extended description so we know what you’ve changed.
  8. Hit the green ‘propose new file’ button.
  9. Review your change and hit the green ‘create pull request’ button.
  10. That’s it - thanks!
  11. Next up we’ll talk to our experts about the change you’ve suggested then evaluate it in our test system.
  12. We’ll get back to you through GitHub with an update to let you know what’s happening and if there’s further input we’d like from you.

@6a6d74
Copy link

6a6d74 commented May 11, 2016

During IPET-MDRD-4 meeting, folks spotted a typo ... so I thought I'd try out your process to fix. At step (8) I had the option to 'commit direct to master' ... I guess that's because I've got commit privilege? Other than that, it was quite simple. I'll share this processing with folks tomorrow, time permitting.

For reference, see wmo-registers/code-manuals#42

@domlowe
Copy link

domlowe commented May 12, 2016

Jeremy just demonstrated this - good work! I questioned whether instead of requiring download from the registry and upload of a new file whether it would be better to use the 'normal' github workflow directly on the .ttl files in github.

i..e find ttl file that is broken,
fork > edit > pull request - discuss, merge

(if you try to edit a file you don't have access to github automatically offers to create a fork)

mike-gormley pushed a commit that referenced this issue Jun 16, 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

No branches or pull requests

3 participants