Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 1, 2026

The refactoring in PR #8402 to use YAML unmarshaling with typed structs obsoleted the parseAllowedReposFromConfig helper function. The allowed-repos field is now parsed directly via struct tags on AllowedRepos []string.

Changes

  • Removed parseAllowedReposFromConfig function definition from pkg/workflow/config_helpers.go
  • Removed function reference from package documentation

The function had zero callers after the typed input refactoring completed.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/user
    • Triggering command: /usr/bin/gh gh api user --jq .login ithub/workflows --jq /bin/sh -json GO111MODULE l/linux_amd64/cg--show-toplevel /bin/sh -c ithub/workflows l/linux_amd64/cgo /usr/bin/git -json GO111MODULE l/linux_amd64/velist git (http block)
    • Triggering command: /usr/bin/gh gh api user --jq .login eam.git -test.v=true /home/REDACTED/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.25.0.linux-amd64/pkg/tool/linux_amd64/colist l GO111MODULE ux-amd64/pkg/toostatus /home/REDACTED/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.25.0.lin--jq -o b/workflows -trimpath /usr/bin/git l main -lang=go1.25 git (http block)
    • Triggering command: /usr/bin/gh gh api user --jq .login /run/containerd/io.containerd.runtime.v2.task/moby/4e49485596b195b0117a6afdf2ec23236d81093a122224c84de6cdd9db1e0c38/init.pid 4e49485596b195b0117a6afdf2ec23236d81093a122224c84de6cdd9db1e0c38 /usr/bin/git 3236d81093a122224c84de6cdd9db1e0c38/log.json -w -buildmode=exe git rev-�� bc9d79513fad3ec9 -extld=gcc /usr/bin/infocmp -json GO111MODULE l/linux_amd64/ve--root infocmp (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/20640936161/job/59272958342#step:7:1

Lint go. See referenced error


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Jan 1, 2026
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix linting errors in Go code Remove unused parseAllowedReposFromConfig after refactoring to typed inputs Jan 1, 2026
Copilot AI requested a review from pelikhan January 1, 2026 16:05
@pelikhan pelikhan marked this pull request as ready for review January 1, 2026 16:07
@pelikhan pelikhan merged commit 2b6785d into copilot/refactor-parsing-functions-typed-inputs Jan 1, 2026
@pelikhan pelikhan deleted the copilot/lint-go-code branch January 1, 2026 16:07
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