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

General improvements #33

Merged
merged 3 commits into from
May 6, 2022
Merged

General improvements #33

merged 3 commits into from
May 6, 2022

Conversation

rustkas
Copy link
Contributor

@rustkas rustkas commented May 3, 2022

  • Tune module descrition (exclude licence topic info from generating documentation by @end EDoc tag.

  • Add documentation targets for generation public and private documenation.

  • -spec. Simplified documentation content. Explicitly specified the name of function parameters in a uniform style.
    Introduced a wider use of the features of EDoc.

  • Provide a description of the data types (-type) that were in the source codes in the generated EDoc documentation. This in the code seems non-obvious and even strange. But please look at the output when generating the documentation. The result - we have documentation for types.

  • cowmachine_req. Fix version documentation type. Arranged the version values in the correct order.

  • rebar.config

    • Add documentation generation and testing profiles.
    • Add option to generate public and private documentation.
    • Moved the work with the dilyzer to a separate profile "check" from the "test" profile. It is more comfortable. It also speeds up work by doing only useful work.
  • EDoc

    • Tune css stylesheet.

Tune module descrition (exclude licence topic info from generating documentation by @EnD EDoc tag.
Add documentation targets for generation public and private documenation.
-spec. Simplified documentation content. Explicitly specified the name of function parameters in a uniform style.
Introduced a wider use of the features of EDoc.

Provide a description of the data types (-type) that were in the source codes in the generated EDoc documentation. This in the code seems non-obvious and even strange. But please look at the output when generating the documentation. The result - we have documentation for types.

cowmachine_req. Fix version documentation type. Arranged the version values in the correct order.

rebar.config
 Add documentation generation and testing profiles.
 Add option to generate public and private documentation.
 Moved the work with the dilyzer to a separate profile "check" from the "test" profile. It is more comfortable. It also speeds up work by doing only useful work.

EDoc
 Tune css stylesheet.
@mmzeeman
Copy link
Member

mmzeeman commented May 6, 2022

Hi Rustkas,

Thanks for contributing. Cowmachine definitely needs to have some extra documentation. Is this PR ready? In the comment you mention an improved css file, but I didn't see it in the PR yet. Anyhow, I will look at your PR later today.

Add description to .gitignore related to style.ccs and overview.edoc.
@rustkas
Copy link
Contributor Author

rustkas commented May 6, 2022

Hi Maas!
Thanks for mentioning about the CSS. I've made adjustments to it to improve the readability of the generated documentation. I didn't make the appropriate adjustments to the .gitignore file.

This PR is ready. Now I am working with next one. I am developing property-based tests for cowmachine.

@rustkas
Copy link
Contributor Author

rustkas commented May 6, 2022

Note related to the conflict into src/cowmachine.erl

dialyzer did warning message related to a piece of this request_1 function. After adding result variable warning was gone.

@mmzeeman
Copy link
Member

mmzeeman commented May 6, 2022 via email

Copy link
Member

@mmzeeman mmzeeman left a comment

Choose a reason for hiding this comment

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

That must have been a lot of work to add the missing specs. The new style of spec definitions look a lot clearer in the code and in the documentation. The zotonic logo is a really nice touch. 👍

@rustkas
Copy link
Contributor Author

rustkas commented May 6, 2022

That must have been a lot of work to add the missing specs. The new style of spec definitions look a lot clearer in the code and in the documentation. The zotonic logo is a really nice touch. +1

Thank you Maas for a positive assessment of my work.
It would be nice to review other Zotonic related pull requests which I already did:

  1. bcrypt

  2. buffalo

  3. dispatch_compiler

  4. poolboy

    1. Documentation improvements
    2. Proper tests
  5. depcache

  6. erlang-certifi

  7. bert.erl

@mmzeeman mmzeeman merged commit ed1aed8 into zotonic:master May 6, 2022
@mworrell
Copy link
Member

mworrell commented May 7, 2022

Released 1.9.0

@rustkas rustkas deleted the fix_doc branch May 9, 2022 14:17
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

3 participants