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

Cleanup edocs. Fixes #213 #214

Merged
merged 3 commits into from Feb 25, 2020
Merged

Cleanup edocs. Fixes #213 #214

merged 3 commits into from Feb 25, 2020

Conversation

seriyps
Copy link
Member

@seriyps seriyps commented Feb 9, 2020

  • Make rebar3 edoc work
  • Add some module-level edocs
  • Add basic doc/overview.edoc
  • Add edoc to travis

Not sure if generated edoc HTMLs are very usefull, but at least it makes it harder to ignore in-the-code documentation during development.

- Make rebar3 edoc work
- Add some module-level edocs
- Add basic doc/overview.edoc
- Add edoc to travis
@seriyps seriyps requested a review from davidw February 10, 2020 17:01
@@ -0,0 +1,71 @@

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this one generated?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I wrote it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems kind of redundant when most of this is in the README - and it's an extra thing to keep up to date, potentially.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, makes sense. The problem is that edoc needs some "overview" page as an index/starting page for edoc. So it's nice to have at least something with a few links to start with.
But I'm agree that it might be better to move all non-basic stuff to the README.
Unfortunately it's not easy to automatically sync readme and edoc.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not really familiar with edocs; could you just link to the README - either locally, or online?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, there is an option - to generate README.md from overview.edoc: https://github.com/uwiger/edown/blob/master/README.md#top-level-readme

But not sure if we should go this way.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, just hyperlink like http://github.com/epgsql/epgsql/README.md can be added. It's also possible to use any HTML there.
But it's nice to have a few links to start with in overview.edoc

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I think my proposal is:
We can assume that this generated edoc "target audience" is not end-users of epgsql, but those who need a guide over the source code for some reason (eg, they want to contribute some feature / implement some custom command or datatype). So, we can make this overview.edoc a starting point for them.
So, there will be no "how to use epgsql", but "how to explore epgsql sources" or, in other words, "internal" instructions. So, I will mention 3 APIs as an entry points and describe what is epgsql_sock, epgsql_command, epgsql_codec and epgsql_binary/epgsql_wire and some helper modules.
So, internal docs - edocs, end-user docs - README, with minimal overlap.
What do you think?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also plan to rework README a bit. Right now it has a lot of historical introductions (how it is different from original @wg version) on the very top, but I would rather move it to the bottom or remove.
Also want to add some examples of basic session examples (like the one I added to overview.edoc) and a couple of best-practices and HOWTOs, but in a separate PR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think avoiding redundancy is a something to aim for. Otherwise, it's easy for these to get out of sync. I do like the idea of breaking the README up a bit.

@seriyps
Copy link
Member Author

seriyps commented Feb 13, 2020

@davidw I updated the overview.edoc to be more "intarnal" docs and made a couple of other minor changes.

@seriyps seriyps requested a review from davidw February 25, 2020 09:53
@seriyps seriyps merged commit 2d4d1ad into epgsql:devel Feb 25, 2020
@seriyps
Copy link
Member Author

seriyps commented Feb 25, 2020

Thanks @davidw !

@seriyps seriyps mentioned this pull request Mar 5, 2020
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.

None yet

2 participants