Skip to content

Rename test/integration-test/, remove stale comment#1

Merged
MarcinZiabek merged 3 commits into
mainfrom
copilot/update-test-folder-name
May 26, 2026
Merged

Rename test/integration-test/, remove stale comment#1
MarcinZiabek merged 3 commits into
mainfrom
copilot/update-test-folder-name

Conversation

Copy link
Copy Markdown

Copilot AI commented May 26, 2026

test was too generic; integration-test accurately reflects that this is an end-to-end native library validation app, not a unit test suite. Also drops a now-outdated inline comment from the copy step in the workflow.

Changes

  • Renamed test/integration-test/
  • Updated all path and cd references in .github/workflows/main.yml
  • Removed stale comment # This now copies qpdf.dll AND all its dependencies into the integration-test folder

@MarcinZiabek
Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

Co-authored-by: MarcinZiabek <9263853+MarcinZiabek@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented May 26, 2026

@copilot resolve the merge conflicts in this pull request

Resolved in 89209a2. The conflict in the "Test build artifacts" step was between integration-test + net6.0 (this branch) and test + net8.0 (main). The resolution keeps integration-test/ from this PR and net8.0 from main.

@MarcinZiabek MarcinZiabek marked this pull request as ready for review May 26, 2026 13:29
Copilot AI review requested due to automatic review settings May 26, 2026 13:29
@MarcinZiabek MarcinZiabek merged commit f408d53 into main May 26, 2026
1 check failed
Copilot AI requested a review from MarcinZiabek May 26, 2026 13:30
Copilot AI review requested due to automatic review settings May 26, 2026 13:52
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