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

Update state.md #7

Merged
merged 1 commit into from
Feb 24, 2022
Merged

Conversation

davidmaxwaterman
Copy link
Contributor

Changed lowercase 'id' to ID to match other use (in html comment).

Changed lowercase 'id' to ID to match other use (in html comment).
@Raynos
Copy link
Contributor

Raynos commented Aug 30, 2021

Hmm, maybe we should change all ID => id

@davidmaxwaterman
Copy link
Contributor Author

I went with: "ID" when used in English, and 'id' when used in code, ie "ID" and id.
Either way, consistency is the key :)

@heapwolf
Copy link
Member

probably in all docs id should be ID, either that otherwise, the reference to id should be surrounded by back-ticks or code-blocks.

@heapwolf heapwolf merged commit dc34b34 into socketsupply:master Feb 24, 2022
@davidmaxwaterman davidmaxwaterman deleted the patch-4 branch February 24, 2022 10:41
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.

3 participants