Skip to content

Fix comment style inconsistency in DBRunInfo_runInfo.cc documentation#240

Merged
rrivera747 merged 3 commits into
jcampos/NewDBSchemafrom
copilot/sub-pr-234
Jan 28, 2026
Merged

Fix comment style inconsistency in DBRunInfo_runInfo.cc documentation#240
rrivera747 merged 3 commits into
jcampos/NewDBSchemafrom
copilot/sub-pr-234

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 27, 2026

Address review feedback from PR #234 to standardize comment style in function documentation.

Changes

  • Changed line 803 in DBRunInfo_runInfo.cc from C-style comment (//) to C++ documentation style (///) to match surrounding documentation comments in the updateRunInfo() function header
/// updateRunInfo
///     Inserts a transition record into the database for a specific run.
///     This is called by the state machine during various transitions to track
///     when state changes occur for a run (e.g., Start, Stop, Pause, Resume, Halt, Error).
-//      Only states Configured and "above" are recorded (aka relevant for run)
+///     Only states Configured and "above" are recorded (aka relevant for run)
///
///     @param runConditionID - For Mu2e, the runConditionID is the database run number...

This stacked PR merges into PR #234.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits January 27, 2026 23:31
Co-authored-by: rrivera747 <107584474+rrivera747@users.noreply.github.com>
Co-authored-by: rrivera747 <107584474+rrivera747@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on code style in new DB schema PR Fix comment style inconsistency in DBRunInfo_runInfo.cc documentation Jan 27, 2026
Copilot AI requested a review from rrivera747 January 27, 2026 23:32
@rrivera747 rrivera747 marked this pull request as ready for review January 28, 2026 00:19
@rrivera747 rrivera747 merged commit 7a7538f into jcampos/NewDBSchema Jan 28, 2026
@rrivera747 rrivera747 deleted the copilot/sub-pr-234 branch January 28, 2026 00:20
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