Skip to content

Conversation

p-linnane
Copy link
Member

Opened Homebrew/brew#20315 in the wrong location, so now trying again in the correct repo.

I've split the commits here for easier review. I alphabetized the ecosystems in each file, and then added Terraform. This is needed for the user-management repo.

Signed-off-by: Patrick Linnane <patrick@linnane.io>
Signed-off-by: Patrick Linnane <patrick@linnane.io>
Copilot

This comment was marked as outdated.

@p-linnane p-linnane requested a review from Copilot July 25, 2025 21:23
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds Terraform support to the Dependabot configuration system used across Homebrew repositories. The changes alphabetize the existing ecosystem configurations and add Terraform as a new supported package ecosystem.

Key changes:

  • Alphabetized ecosystem entries in the shared configuration file
  • Added Terraform ecosystem support with .terraform.lock.hcl as the target file
  • Added Terraform package ecosystem configuration to the Dependabot template

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/actions/sync/shared-config.rb Alphabetized ecosystem cases and added Terraform with .terraform.lock.hcl file mapping
.github/actions/sync/dependabot.template.yml Alphabetized package ecosystems and added Terraform configuration block

@p-linnane p-linnane force-pushed the dependabot-updates branch 2 times, most recently from 35ee759 to 7a21ad2 Compare July 25, 2025 21:35
Signed-off-by: Patrick Linnane <patrick@linnane.io>
Signed-off-by: Patrick Linnane <patrick@linnane.io>
@p-linnane p-linnane force-pushed the dependabot-updates branch from 7a21ad2 to 471fe5c Compare July 25, 2025 21:36
@p-linnane
Copy link
Member Author

Accidentally fixed up a commit incorrectly. Should be good now. Sorry for all the force pushes 😅

@p-linnane p-linnane merged commit 17087f4 into main Jul 26, 2025
28 checks passed
@p-linnane p-linnane deleted the dependabot-updates branch July 26, 2025 15:45
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