[Deprecate] Billing for GitHub Actions is now supported in enhanced billing only.

[Manas23601]

Description

The BillingActions endpoint has been moved to the enhanced billing endpoint. Previously, it had its own separate endpoint as follows:

# https://docs.github.com/en/enterprise-cloud@latest/rest/billing/billing?apiVersion=2022-11-28#get-github-actions-billing-for-an-organization
/orgs/{org}/settings/billing/actions

Now:

curl -L -H "Accept: application/vnd.github+json" -H "Authorization: Bearer <TOKEN>" -H "X-GitHub-Api-Version: 2022-11-28" https://github.com/api/orgs/<ORG>/settings/billing/actions

# Returns:
{
  "message": "This endpoint has been moved.",
  "documentation_url": "https://gh.io/billing-api-updates-org",
  "status": "410"
}

Changes

Remove unuseable structs and tests for actionsBilling

[Manas23601]

@gmlewis , I had requested this feature earlier - #3592.

I was testing it today and I realized actionsBilling method is no longer valid.

I also made the return types in UsageReport to be pass by value instead of (Pointers) pass by reference. Correct me if I am wrong, input needs to be pass by reference, only if changes to the obj need to be persistent, which is not the case here

[Manas23601]

Nvm, changing stuff in Usage report breaks the code.

type UsageReportOptions struct {
	// If specified, only return results for a single year. The value of year is an integer with four digits representing a year. For example, 2025.
	// Default value is the current year.
	Year *int `url:"year,omitempty"`

	// If specified, only return results for a single month. The value of month is an integer between 1 and 12.
	// If no year is specified the default year is used.
	Month *int `url:"month,omitempty"`

	// If specified, only return results for a single day. The value of day is an integer between 1 and 31.
	// If no year or month is specified, the default year and month are used.
	Day *int `url:"day,omitempty"`

	// If specified, only return results for a single hour. The value of hour is an integer between 0 and 23.
	// If no year, month, or day is specified, the default year, month, and day are used.
	Hour *int `url:"hour,omitempty"`
}

Why do we need pointers here and not in ListOptions

// github/github.go
type ListOptions struct {
	// For paginated result sets, page of results to retrieve.
	Page int `url:"page,omitempty"`

	// For paginated result sets, the number of results to include per page.
	PerPage int `url:"per_page,omitempty"`
}

// /github/billing.go
func (s *BillingService) GetAdvancedSecurityActiveCommittersOrg(ctx context.Context, org string, opts *ListOptions) (*ActiveCommitters, *Response, error) {
	u := fmt.Sprintf("orgs/%v/settings/billing/advanced-security", org)
	u, err := addOptions(u, opts)
	if err != nil {
		return nil, nil, err
	}
.
.
.

[gmlewis]

Why do we need pointers here and not in ListOptions

Right... we don't absolutely need the pointers here.
We try to keep the repo as self-consistent as possible, and whenever we use omitempty, we try to use pointers for JSON fields, but since these are url tags, we really could change them to values (which would obviously be a breaking API change).