Skip to content

Add XML documentation for Time.Create parameter descriptions#256

Closed
Copilot wants to merge 3 commits intofeature/nrlmsisfrom
copilot/sub-pr-255
Closed

Add XML documentation for Time.Create parameter descriptions#256
Copilot wants to merge 3 commits intofeature/nrlmsisfrom
copilot/sub-pr-255

Conversation

Copy link

Copilot AI commented Dec 24, 2025

Addresses incomplete XML documentation for Time.Create(int year, int doy, double sec) method parameters that were missing descriptions.

Changes

  • Added parameter descriptions with valid ranges:
    • year: Calendar year (0 defaults to 2000)
    • doy: Day of year (1-366)
    • sec: Seconds within the day (0-86399)
/// <summary>
/// Create time from year, day of year, and seconds of day
/// </summary>
/// <param name="year">Calendar year (0 defaults to 2000)</param>
/// <param name="doy">Day of year (1-366)</param>
/// <param name="sec">Seconds within the day (0-86399)</param>
/// <returns></returns>
public static Time Create(int year, int doy, double sec)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update to address review feedback on feature/nrlmsis Add XML documentation for Time.Create parameter descriptions Dec 24, 2025
@sylvain-guillet sylvain-guillet deleted the copilot/sub-pr-255 branch December 24, 2025 13:37
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