cc @muescha

Hi @tesseralis could you please assign this to me? I have been looking to do some open source contributions.

And I am somewhat confused on this issue, would you be able to provide more context for this?

@robintom since this issue is like an "umbrella" issue: the best is if you take only one small change at a time (for example put one package in code fences). this is more easy to approve a pull request.

anounce here which change you will do, so others can take an other one.

@muescha Hey I will be working on this issue. In particular this part:

• put filenames like .htaccess in code blocks

Should I open a PR for this particular part and then reference this issue?

See my PR below. Is that alright? (It is my first time contributing so let me know :))

@FocalChord great! - i updated the issue text

@muescha So for each other sub-issue do I make a seperate branch and then seperate PR?

Also when does the PR get merged in?

Hint: searching for text which is not already in code fences in files with RegEx (and File Mask *.md*)

[^`]some-thing[^`] 
[^`]\.htaccess[^`] // with dots

BTW: in WebStorm it shows you a section "Usage in comments" if the text is already in code blocks :)

yes - new branch and new PR

it takes some time to get merged, just be patient. the Gatsby team has to approve each PR and with smaller steps this is more easy to check :)

@muescha Awesome! Thanks for this :) I'll work on the 2nd, 3rd, 4th bulletpoints soon.

Hello @muescha and @FocalChord, I'm new here and I'd like to help. Since the list is being tackled one bullet point at a time, I was thinking I could start on bullet point no. 5.

some image alt text in tutorial pages are just file names (e.g. )

If I understand the task correctly, there are two parts to this:

• Delete "with-layout2" and other similar lines from dictionary.txt
• Go through the docs and give better alt text for those images

Do I have the right idea? But before then, is this even a ticket that's looking for help, or should I wait for the next one?

@meyian Yup! We'd want to add better alt text to the images in the tutorial.

i would suggest (for expample for type 1) a PR should only remove one line per dictionary (or like in #25339 similar ones) so PR are small scoped and approved fast.

somehow it not catches all places?

i run the lint for #25355 but in not catches for the NPM in:

update: this is because the heading is marked as SourceNode, i think because of the slashes:

├─29 ParagraphNode
│   └─0 SentenceNode
│       └─0 SourceNode "Backbone.js/Node.js/NPM"

but this get catched:

Hello! I wouldn't mind helping out too! I'd like to tackle 6 but I'm a bit confused about which imported components are being referred to, is it all imported components or only ones involving remark-mdx?

if I understand this correctly, for point 7 we need to edit the dictionary.txt and spilt the text characters from the numbers. Similarly for point 8, we need to split the words with dashes to separate individual words.
Can I work on this :)

7. try out if this working like this.

but:

6. it is a more general change to the system and i think the hardest one...

(edit: because Markdown parser hier in the comment changed the numbers from 6 to 8 if it is one line behind the seven)

okay, I'll start working on these.

an other bug:

the spell checker not reach the table content - for example i forbid *AgilityCMS in dictionary, but it not found it in a title in a table cell:

ok - it seems that a table cell is never checked because of:
https://github.com/syntax-tree/mdast-util-to-nlcst/blob/a05338c5aa287885cfa473b4099d51ca42629c64/index.js#L10

var ignore = ['table', 'tableRow', 'tableCell']

i will investigate how it can be done ....

suggested solution:

Maybe using remark-rehype and then rehype-retext is better?

i get it done, will file a PR in next days...

@muescha I have done a few more issues above :)

@FocalChord thanks a lot :)

Hello, I had two questions:

@muescha I realize you renamed my pull request. Is getting the right format for pull requests something I can learn?

Then, I pushed some changes to docs/tutorial/part-three/index.md and the build failed, because

warning  `uncentered` is misspelt; did you mean `uncensored`?  uncentered  retext-spell

Would the fix for this be as simple as adding the word "uncentered" to dictionary.txt? Or how else should I address this?

PR Title

there are some recommendations for PR titles:

https://www.gatsbyjs.org/contributing/managing-pull-requests/#commit-and-pr-title

i unified a little bit the titles of related PRs of this issue to make it more easy to identify them for the review team.

Uncentered

yes i see it is a new unknown word which you added to the PR, if you are sure that is not a spelling error (like in this case) and not a file name/function name/variable name/package name (put them in code fences) then you can add it to the dictionary.txt.

I added uncentered to the dictionary as discussed above to keep that PR moving, and it's merged now. Is there anything else left to do on this issue, or can we close it?

@marcysutton there are much more entries to clean up in the dictionary.txt, i only placed "current activities to the umbrella issue, because there are 2194 lines in dictionary to check...

Hi, I am a beginner and want to contribute here. I am confused. Can someone guide me?

@Hina-softwareEngineer you are welcome

Overview

you can start with the docs about contributing and the section Repo Setup

contribution to this issue

• for this issue you look for only one entry in the dictionary.txt
• remove this wrong entry or fix this one entry.
• then run the yarn script lint:docs and see where there are the problems and then fix it (see also Docs Contributions)

@gatsbyjs/learning did i forget an instruction?

I just created a PR which removed several words at once, because the first word "allfile-query" appeared in two docs files. In cleaning up the alt text of these files, they had three other wrong dictionary.txt words in common, so I thought it made the most sense to clean up both these files at once and remove any words from dictionary.txt that no longer exist in the docs after this cleanup. I hope I had the right idea, but feel free to let me know if this PR needs splitting up.

@tesseralis: I wanted to edit the Current Activities section in the main comment above and add the new words I was working on, but I wasn't sure if having a PR opened means I check the box next to my entry, or that's only for if the work has been merged in. How does it work?

(Edit: Also, in my PR, I used the "official" camelCase spelling of the word allFile, and now that the linter has run, it's complaining about allFile not being "allfile". What would be the ideal situation in this case. Should I rename allFile to allfile, or would a new dictionary entry be needed to handle the camelCase spelling?)

very good!

yes you can edit it :)

• an empty checkbox means it is not merged.
• checked entries are merged into the repo

(i just followed the other "Umbrella" Issues (like [Umbrella] TypeScript Migration (Help wanted!) #21995), but to list all dictionary entries here for claiming a word would be a little to much...)

allFile

• i see are all the allFile in your PR are code fences

i tested:

• this is a "bug" or a feature? - in descriptions the code fences are not recognized and thats why they are not ecluded from checking :/
• but i think it is a feature, because the alt text can not "enhanced" by markdown stuff (in the github preview of the file the code fences are removed from the image alt text)

i would suggest to reword the descriptions so that code fences are not needed
/cc @tesseralis what you suggest?

@muescha Okay, no problem. I reworded the alt texts to remove the code fences and the references to "allFile".

With PR #26304, all the incorrect alt texts in dictionary.txt have been replaced, I believe. I used the regex !\[.*-.*\]\(.*\) to search through the repo for all kebab-case filenames in alt texts, and all the entries that appeared in dictionary.txt have been removed in the various PRs.