Skip to content

Move UpdatesSiteInformation trait to SiteService#3856

Merged
josephsnyder merged 1 commit into
Kitware:masterfrom
williamjallen:site-service
Jul 21, 2026
Merged

Move UpdatesSiteInformation trait to SiteService#3856
josephsnyder merged 1 commit into
Kitware:masterfrom
williamjallen:site-service

Conversation

@williamjallen

Copy link
Copy Markdown
Collaborator

In the time since the UpdatesSiteInformation trait was created, CDash has developed a service layer which is a more appropriate location for these kinds of utility methods. This PR moves the contents of the UpdatesSiteInformation trait to a static method in a new SiteService.

In the time since the `UpdatesSiteInformation` trait was created, CDash has developed a service layer which is a more appropriate location for these kinds of utility methods.  This PR moves the contents of the `UpdatesSiteInformation` trait to a static method in a new `SiteService`.

@josephsnyder josephsnyder 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.

LGTM!

@josephsnyder
josephsnyder added this pull request to the merge queue Jul 21, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 21, 2026
@josephsnyder
josephsnyder added this pull request to the merge queue Jul 21, 2026
Merged via the queue into Kitware:master with commit 924d24f Jul 21, 2026
19 of 21 checks passed
@williamjallen
williamjallen deleted the site-service branch July 21, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants