Skip to content

Remove Project.exec usage#2674

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/remove-project-exec-usage
Jun 6, 2025
Merged

Remove Project.exec usage#2674
0xnm merged 1 commit into
developfrom
nogorodnikov/remove-project-exec-usage

Conversation

@0xnm

@0xnm 0xnm commented May 26, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Project.exec usage is deprecated since Gradle 8.11 and this API will be removed in Gradle 9.0.

This PR switched to the ExecOperations.exec usage to avoid deprecation warning in the build logs.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm marked this pull request as ready for review May 26, 2025 15:00
@0xnm
0xnm requested review from a team as code owners May 26, 2025 15:00
@0xnm
0xnm marked this pull request as draft May 26, 2025 15:04
xgouchet
xgouchet previously approved these changes May 26, 2025
@0xnm
0xnm force-pushed the nogorodnikov/remove-project-exec-usage branch from 9b11506 to 1f21b22 Compare June 6, 2025 10:17
@0xnm
0xnm force-pushed the nogorodnikov/remove-project-exec-usage branch from 1f21b22 to b28029e Compare June 6, 2025 10:24
@0xnm
0xnm marked this pull request as ready for review June 6, 2025 10:30
@0xnm
0xnm requested a review from xgouchet June 6, 2025 10:30
@codecov-commenter

codecov-commenter commented Jun 6, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.89%. Comparing base (60a1945) to head (b28029e).
⚠️ Report is 2121 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2674      +/-   ##
===========================================
- Coverage    69.96%   69.89%   -0.07%     
===========================================
  Files          822      822              
  Lines        30639    30639              
  Branches      5158     5158              
===========================================
- Hits         21434    21413      -21     
- Misses        7765     7784      +19     
- Partials      1440     1442       +2     

see 34 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0xnm
0xnm merged commit e28a89e into develop Jun 6, 2025
25 checks passed
@0xnm
0xnm deleted the nogorodnikov/remove-project-exec-usage branch June 6, 2025 13:55
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.

4 participants