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

doc: add link to examples #1719

Merged
merged 2 commits into from
Mar 1, 2024
Merged

doc: add link to examples #1719

merged 2 commits into from
Mar 1, 2024

Conversation

rklec
Copy link
Contributor

@rklec rklec commented Mar 1, 2024

As a new user of this library, looking into the Readme and docs for the first time, I struggled to find the amazing(!) complete examples provided in the documentation.

IMHO thus giving a short link there, that there are more examples, is very helpful!

As a new user of this library, looking into the Readme and docs for the first time, I struggled to find the amazing(!) complete examples provided in the documentation.

IMHO thus giving a short link there, that there are more examples, is very helpful!
@jstedfast
Copy link
Owner

Those examples are designed to be included in the API Reference docs and aren't really designed as standalone examples.

It would be more appropriate to link to the samples directory, but I don't really see the point of that since it's in the file listing at the top of the GitHub page.

Improve the wording and link to Documentation/Examples
@jstedfast
Copy link
Owner

How about this modified verbiage/link instead?

@jstedfast jstedfast merged commit 51462a1 into jstedfast:master Mar 1, 2024
4 of 5 checks passed
rklec added a commit to rklec/MailKit that referenced this pull request Mar 1, 2024
@rklec
Copy link
Contributor Author

rklec commented Mar 1, 2024

Ah yeah, also found the /samples directory now, but the other examples have more examples. I would personally link to both, as they are short examples and bigger ones.

But I saw you've merged it in 51462a1.

How about this modified verbiage/link instead?

Don't know, do you like that more? Usually that's not necessary AFAIK as GitHub automatically expands links.

Created #1720 for the samples dir now.

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