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

Generating docs with ExDoc instead of EDoc #308

Merged
merged 3 commits into from
Feb 14, 2022
Merged

Generating docs with ExDoc instead of EDoc #308

merged 3 commits into from
Feb 14, 2022

Conversation

cw789
Copy link
Contributor

@cw789 cw789 commented Feb 12, 2022

Clean new PR replacing #290.


Additionally this PR removes code meant for < OTP 21.
Additionally this PR introduces format checking in CI.
This part is probably opinionated if task should be allowed to fail
until every maintainer is used to check the format beforehand.

I've added the usage of ex_doc & fmt to the Makefile as a kind of documentation.
For myself it is not 100% clear how I could make sure I use the latest rebar3 plugin as they are not anywhere pinned.
-> Upgrading Plugins
This also means I'm not sure how reproducible the CI is regarding the plugins.

All the best.

Uses the rebar3 plugin `rebar3_ex_doc`.
Can be used by `rebar3 ex_doc`.
@seriyps
Copy link
Collaborator

seriyps commented Feb 12, 2022

I think plugins can be pinned same way as dependencies:

{project_plugins, [
    {erlfmt, "1.0.0"},
    {rebar3_proper, "0.12.1"}
]}.

Don't know why I haven't done it initially…

Copy link
Collaborator

@seriyps seriyps left a comment

Choose a reason for hiding this comment

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

Thanks!

- name: Build EDoc
run: make docs
- name: Generate docs by ExDoc
run: rebar3 ex_doc
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The run command could have stayed the same, now as I also changed the Makefile.

@mworrell mworrell merged commit c6f25a7 into gen-smtp:master Feb 14, 2022
@mworrell
Copy link
Collaborator

@cw789 Thanks!

@cw789 cw789 deleted the rebar3_ex_doc branch February 14, 2022 09:08
@cw789
Copy link
Contributor Author

cw789 commented Feb 14, 2022

Thank you too. Hope it doesn't bring any burden to the one who do releases.

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