Skip to content

add integration tests - #89

Open
katbyte wants to merge 8 commits into
mainfrom
kt/integration-tests
Open

add integration tests#89
katbyte wants to merge 8 commits into
mainfrom
kt/integration-tests

Conversation

@katbyte

@katbyte katbyte commented Jul 27, 2026

Copy link
Copy Markdown
Owner

No description provided.

@katbyte
katbyte force-pushed the kt/integration-tests branch from e70e30b to a3851a7 Compare August 3, 2026 06:51
@katbyte

katbyte commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	.github/workflows/test.yaml
#	lib/gh/file.go
#	lib/tc/request.go
#	makefile

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Merged origin/main into the branch and resolved all conflicts:

  • .github/workflows/test.yaml: Kept the integration test paths from the branch, updated to use actions/checkout@v7 and actions/setup-go@v7 with go-version-file from main.
  • lib/gh/file.go: Kept RawURL support for mock servers in integration tests; fixed variable declaration (content, err :=).
  • lib/gh/gh.go: Updated Repo.NewClient() to use github.WithEnterpriseURLs() instead of direct client.BaseURL assignment (which is no longer a field in go-github v89).
  • lib/tc/request.go: Kept the baseURL() helper that supports custom URL schemes (needed for integration tests pointing at local mock servers).
  • makefile: Kept the integration test target; merged .PHONY declarations from both sides.

Commit: 9d66447

@katbyte katbyte closed this Aug 3, 2026
@katbyte katbyte reopened this Aug 4, 2026
@katbyte

katbyte commented Aug 4, 2026

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	cli/prs.go
#	makefile

Co-authored-by: katbyte <1638467+katbyte@users.noreply.github.com>

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Merged the latest origin/main commits and resolved conflicts:

  • cli/prs.go: Trivial whitespace conflict resolved.
  • makefile: Adopted the new structure/help comments from main (##@ sections, help target, updated .PHONY); kept the integration test target from this branch.

Commit: 0a033c6

@mstroob mstroob left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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