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

Small documentation improvements #504

Merged
merged 3 commits into from
Sep 8, 2024
Merged

Conversation

imphil
Copy link
Contributor

@imphil imphil commented Aug 18, 2024

While setting up the GitHub Action provided by this project I came across some minor papercuts in the documentation that confused me initially.

See the individual commit messages for details.

  • Documentation: Fix typos
  • Docs: Use main branch links to source code
  • Docs: Improve consistency in prTitle documentation

imphil added 3 commits August 19, 2024 00:25
The main branch evolves, and the documentation should evolve with it.
Link to TypeScript interfaces and code samples on the main branch to
avoid confusing users with outdated information.

The documentation can still go out-of-date, of course. Mitigate that a
bit by avoiding to link exact line numbers within a file, since they are
most likely to change.
@imphil
Copy link
Contributor Author

imphil commented Aug 18, 2024

Ah, I just saw I fixed the same issue as #503 (among other things). I'll rebase when the other PR goes in first.

@sorenlouv sorenlouv enabled auto-merge (squash) September 8, 2024 16:25
@sorenlouv
Copy link
Owner

Thanks for the contribution!

@sorenlouv sorenlouv disabled auto-merge September 8, 2024 16:25
@sorenlouv sorenlouv merged commit d39f0b2 into sorenlouv:main Sep 8, 2024
@sorenlouv sorenlouv mentioned this pull request Sep 8, 2024
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.

2 participants