Skip to content

Remove unused context import from remote fetch integration test#43348

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-unused-context-import
Closed

Remove unused context import from remote fetch integration test#43348
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-unused-context-import

Conversation

Copilot AI commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

The parser remote fetch integration test referenced t.Context() but still imported the standard-library context package directly. That left pkg/parser/remote_fetch_integration_test.go failing to compile in CI with an unused-import error, breaking the affected integration jobs.

  • Scope

    • Remove the unused context import from pkg/parser/remote_fetch_integration_test.go.
  • Effect

    • Restore compilation of the parser integration test package.
    • Unblock CI jobs that exercise remote fetch and unauthenticated add flows.
  • Code

    import (
    	"strings"
    	"testing"
    
    	"github.com/cli/go-gh/v2/pkg/api"
    	"github.com/github/gh-aw/pkg/errorutil"
    	"github.com/stretchr/testify/assert"
    	"github.com/stretchr/testify/require"
    )

Copilot AI changed the title [WIP] Fix unused context import in remote fetch integration tests Remove unused context import from remote fetch integration test Jul 4, 2026
Copilot AI requested a review from pelikhan July 4, 2026 08:41
@pelikhan pelikhan closed this Jul 4, 2026
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.

CI: Integration Parser Remote Fetch tests break main due to unused context import

2 participants