Skip to content

Improve and repair some docs#142

Merged
steve-downey merged 2 commits intobemanproject:mainfrom
steve-downey:fixdocs
Sep 22, 2025
Merged

Improve and repair some docs#142
steve-downey merged 2 commits intobemanproject:mainfrom
steve-downey:fixdocs

Conversation

@steve-downey
Copy link
Copy Markdown
Member

Wrong docs were applied to emplace, and added details to other functions.

Wrong docs were applied to `emplace`, and added details to  other functions.
Comment thread include/beman/optional/optional.hpp Outdated
* @brief Nullopt assignment operator.
*
* @return optional&
*
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
*
*

Comment thread include/beman/optional/optional.hpp Outdated
* @tparam F
* @param f
* @return auto
*
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
*
*

Comment thread include/beman/optional/optional.hpp Outdated
*
* @details
* The return type is the same as \tcode{std::invoke_result_t<F, T&>},
* but wrapped in an optional. The function \p f must return an optional type.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
* but wrapped in an optional. The function \p f must return an optional type.
* but wrapped in an optional. The function \p f must return an optional type.

Comment thread include/beman/optional/optional.hpp Outdated
* @tparam F
* @param f
* @return optional<std::invoke_result_t<F, T&>>
*
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
*
*

Comment thread include/beman/optional/optional.hpp Outdated
* @return optional
* @details
* The return type is the same as the return type of \p f, which must
* return an optional type.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
* return an optional type.
* return an optional type.

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 16, 2025

Pull Request Test Coverage Report for Build 17786715427

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.31%

Totals Coverage Status
Change from base Build 17766602718: 0.0%
Covered Lines: 2446
Relevant Lines: 2463

💛 - Coveralls

Fix whitespace issues.
@steve-downey steve-downey merged commit ed2413e into bemanproject:main Sep 22, 2025
11 checks passed
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