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

Improve documentation #116

Closed
bobluppes opened this issue Sep 30, 2023 · 14 comments
Closed

Improve documentation #116

bobluppes opened this issue Sep 30, 2023 · 14 comments
Assignees
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed no-issue-activity

Comments

@bobluppes
Copy link
Owner

This is an open-ended ticket and we welcome multiple PRs.

In general, it would be nice to go over our documentation in the following locations:

  • README.md
  • In-code documentation on:
    • The core data structures in include/graaflib/graph.h, include/graaflib/edge.h, and include/graaflib/types.h
    • The algorithms in include/graaflib/algorithm/**/*.h
  • The user-facing algorithm documentation in docs/docs/algorithms/**/*.md. This documentation can also be viewed here.

In particular it would be great if we can fix any spelling/grammar issues, as well as clarify anything which might be unclear to you on the documentation.

@bobluppes bobluppes added documentation Improvements or additions to documentation help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest labels Sep 30, 2023
@bobluppes bobluppes pinned this issue Sep 30, 2023
@Harsh-710
Copy link

Hey @bobluppes, I want to work on this issue. Can you assign it to me?

@LinhNguyenLe2109
Copy link
Contributor

Hello @bobluppes, I have knowledge about graph and edge, can I work on that section?

@bobluppes
Copy link
Owner Author

Hi @LinhNguyenLe2109 @Harsh-710, welcome to Graaf!

Since we accept multiple PRs on this issue I will assign it to the both of you! To avoid potential merge conflicts, what do you think about @Harsh-710 working on the in-code documentation while @LinhNguyenLe2109 takes a look at the readme and user facing docs?

We can swap this afterwards as well, just trying to avoid merge conflicts :)

Let me know what you think

@SusovanDas19
Copy link

hi sir, can I do this?

@LinhNguyenLe2109
Copy link
Contributor

Hello @bobluppes, yeah that's a good idea, I find the doc really hard to get a hang of the project and want more information to it so people can find it easier to use. I'll try to work with the readme file first, if I don't understand anything, I'll reach out to you on Discord

@ayushmankoley
Copy link

Hey would love to contribute
The core data structures in include/graaflib/graph.h, include/graaflib/edge.h, and include/graaflib/types.h

@LinhNguyenLe2109
Copy link
Contributor

Hello @bobluppes, I believe I can only do with the readme file. I sent you a PR for that one. The set up process for C++ using VS code really takes a long time though.

@ankitrout2903
Copy link

Hey assign me this issue

@bobluppes
Copy link
Owner Author

Hi @SusovanDas19 @ayushmankoley @ankitrout2903, are you still up for this.
We are accepting multiple PRs on this issue, so if you want feel free to open a PR for any improvements you may have :)

@ayushmankoley ayushmankoley removed their assignment Oct 9, 2023
@Harsh-710 Harsh-710 removed their assignment Oct 9, 2023
@github-actions
Copy link
Contributor

Stale issue message

@GovindaRohith
Copy link

Can you assign me to this?

@DaryaYuk
Copy link

DaryaYuk commented Nov 7, 2023

@bobluppes Hi! I am a technical writer and I would like to improve the Readme file. If you still need help, please assign me to work on this issue

@bobluppes
Copy link
Owner Author

@bobluppes Hi! I am a technical writer and I would like to improve the Readme file. If you still need help, please assign me to work on this issue

Hi @DaryaYuk and welcome to Graaf!
Of course, new contributions are always welcome! Congrats on the first PR already, I will have a look shortly

Copy link
Contributor

Stale issue message

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed no-issue-activity
Projects
None yet
Development

No branches or pull requests

8 participants