Skip to content

Replace TBD XML doc placeholders in Akka.Actor.Scheduler#8119

Merged
Aaronontheweb merged 3 commits intoakkadotnet:devfrom
MattKotsenas:docs/scheduler
Apr 23, 2026
Merged

Replace TBD XML doc placeholders in Akka.Actor.Scheduler#8119
Aaronontheweb merged 3 commits intoakkadotnet:devfrom
MattKotsenas:docs/scheduler

Conversation

@MattKotsenas
Copy link
Copy Markdown
Contributor

Changes

Replace TBD XML doc placeholders in the Akka.Actor.Scheduler namespace with useful documentation. Five files updated:

  • HashedWheelTimerScheduler.cs - documented constructor; used <inheritdoc/> for 8 overrides that delegate to the base class
  • IAdvancedScheduler.cs - documented the interface summary describing Action-based scheduling via IScheduler.Advanced
  • IScheduler.cs - documented the interface summary referencing ITellScheduler, ITimeProvider, and ActorSystem.Scheduler
  • ITimeProvider.cs - documented MonotonicClock and HighResMonotonicClock properties
  • SchedulerBase.cs - documented constructor, 6 abstract scheduling methods, and 2 validation helpers

Partial fix for #3435.

Checklist

Copy link
Copy Markdown
Member

@Aaronontheweb Aaronontheweb left a comment

Choose a reason for hiding this comment

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

LGTM

@Aaronontheweb Aaronontheweb enabled auto-merge (squash) April 13, 2026 19:33
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines:
Successfully started running 1 pipeline(s).

@Aaronontheweb Aaronontheweb disabled auto-merge April 23, 2026 14:20
@Aaronontheweb Aaronontheweb merged commit 2b8aaea into akkadotnet:dev Apr 23, 2026
9 of 12 checks passed
@MattKotsenas MattKotsenas deleted the docs/scheduler branch April 23, 2026 15:57
Aaronontheweb pushed a commit that referenced this pull request Apr 24, 2026
Co-authored-by: Gregorius Soedharmo <arkatufus@yahoo.com>
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