Skip to content

Add Doxygen config - #584

Merged
urfeex merged 2 commits into
UniversalRobots:masterfrom
urfeex:doxygen
Sep 30, 2026
Merged

urfeex merged 2 commits into
UniversalRobots:masterfrom
urfeex:doxygen

Conversation

@urfeex

@urfeex urfeex commented Sep 30, 2026

Copy link
Copy Markdown
Member

Add a doxygen config to the project. I've had that laying around for a while always getting in my way of unstaged files. It's not fully polished and no workflow exists to build it automatically. But it also shouldn't hurt having this file checked in.

@urfeex
urfeex requested review from a team and a balanced review from Copilot September 30, 2026 07:35
@urfeex urfeex added the documentation Improvements or additions to documentation label Sep 30, 2026

Copilot AI left a comment

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.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@cursor

cursor Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only change with no runtime or library behavior impact.

Overview
Checks in a Doxygen 1.9.8 Doxyfile so API docs can be generated locally without keeping an unstaged config around. The project is named Universal Robots Client Library; input is src, include/ur_client_library, and README.md (recursive), with README.md as the main page. HTML goes to doc_out/doxygen, with source browsing and reference/called-by relations enabled and preprocessor include path include.

.gitignore now ignores doc_out/* so generated output is not committed. There is still no CMake or CI step to build docs automatically—the PR only adds the config file.

Reviewed by Cursor Bugbot for commit c8c6ebe. Bugbot is set up for automated code reviews on this repo. Configure here.

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.72%. Comparing base (2efb77b) to head (c8c6ebe).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #584      +/-   ##
==========================================
- Coverage   89.61%   87.72%   -1.89%     
==========================================
  Files           2        2              
  Lines         443      440       -3     
==========================================
- Hits          397      386      -11     
- Misses         46       54       +8     
Flag Coverage Δ
python_scripts 75.90% <ø> (ø)
start_ursim 90.47% <ø> (-2.31%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@urrsk urrsk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Agree

@mergify

mergify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@urfeex
urfeex merged commit d00cd65 into UniversalRobots:master Sep 30, 2026
53 of 66 checks passed
@urfeex
urfeex deleted the doxygen branch September 30, 2026 10:31
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants