Skip to content

Bump version of http4k #2791

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jBorkowska wants to merge 71 commits into
base: master
Choose a base branch
from
Open

Bump version of http4k #2791

jBorkowska wants to merge 71 commits into from

Conversation

@jBorkowska
Copy link
Collaborator

@jBorkowska jBorkowska commented Nov 17, 2025 •
edited
Loading

This change has implications:

  • projects using this patrol version have to upgrade kotlin to 2.2.0 and bump minSdk to 26
  • android setup in gradle changes - a packaging section is added
  • building tests will show a lot of native android warnings in verbose. To get rid of them, AGP in the project has to be 8.10

Krzysztof Mamak and others added 30 commits September 10, 2025 12:41
Change default test directory to patrol_test, allow overriding test d…
cfbbfb4 
…irectory
Update default test directory values
eeceedb
Update documentation
1865254
Update CHANGELOG
9961da2
Improve findAllTests documentation
3a2fa74
Add missing test cases
5def0f4
Change new patrol_cli version
f24382d
Delay pubspec reading
aa3a18e
Update CHANGELOGs
0d1199c
Update CHANGELOG, restore versions
56b5b95
Fix bundle test file paths, update tests, update CHANGELOGs
aaf6ab1
Restore patrol_cli version constant
74ce6ae
Fix formatting
e2bd4af
Change test directory migration option description
c52a8e9
Extend Configuring test directory documentation section
f3e67f5
Correct "Migrating from integration_test directory" sections
559b464
Update CHANGELOGs
925bff6
Bump packages versions
291c595
Update CHANGELOGs
1bfb9e7
Update integration_test references, fix tests
0a27dd5
Remove unnecessary import
d02fb5b
@KrzysztofMamak
Merge pull request #2728 from leancodepl/feat/change-default-patrol-t…
b64c9f3 
…est-directory

Add configurable test directory support
Replace clearing permissions with uninstalling the app, update flags
13b92af
Fix uninstallApp function
0a2cba6
Refactor uninstallApp function
899fdd0
Prevent from uninstalling the app before the first test
027acad
Change uninstall app option to clear package data
dc32560
Change the way of uninstalling the app
60f86d2
Remove unnecessary function
7cb7efc
Adjust uninstallApp function to support other languages
464dc52
@jBorkowska jBorkowska force-pushed the experiment/http-proxy branch from 59016af to a5e5170 Compare November 18, 2025 14:41
@jBorkowska jBorkowska marked this pull request as ready for review November 18, 2025 15:52
@jBorkowska jBorkowska force-pushed the experiment/http-proxy branch from 80988da to b6a9751 Compare November 18, 2025 16:00
@github-actions github-actions bot added docs Concerns doc comments, READMEs, or docs package: patrol_cli Related to the patrol_cli package labels Nov 18, 2025
@jBorkowska jBorkowska changed the base branch from master to master-v4 November 18, 2025 16:00
@jBorkowska jBorkowska removed the package: patrol_cli Related to the patrol_cli package label Nov 18, 2025
@github-actions github-actions bot added package: adb Related to the adb package package: patrol_cli Related to the patrol_cli package labels Nov 20, 2025
@jBorkowska jBorkowska force-pushed the experiment/http-proxy branch from 814d321 to 6298953 Compare November 20, 2025 14:25
@jBorkowska jBorkowska removed package: adb Related to the adb package package: patrol_cli Related to the patrol_cli package labels Nov 20, 2025
@piotruela piotruela force-pushed the master-v4 branch 2 times, most recently from 3129542 to 3a4dd2e Compare December 5, 2025 13:26
Base automatically changed from master-v4 to master December 8, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pdenert pdenert Awaiting requested review from pdenert pdenert is a code owner

@piotruela piotruela Awaiting requested review from piotruela piotruela is a code owner

@mateuszwojtczak mateuszwojtczak Awaiting requested review from mateuszwojtczak mateuszwojtczak is a code owner

@fylyppo fylyppo Awaiting requested review from fylyppo fylyppo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

docs Concerns doc comments, READMEs, or docs package: patrol Related to the patrol package (native automation, test bundling)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jBorkowska @KrzysztofMamak @zoskar