Skip to content

Handle relative ext/ paths and test the windows installer#2559

Merged
bwoebi merged 3 commits intomasterfrom
bob/relative-ext-path
Mar 7, 2024
Merged

Handle relative ext/ paths and test the windows installer#2559
bwoebi merged 3 commits intomasterfrom
bob/relative-ext-path

Conversation

@bwoebi
Copy link
Copy Markdown
Collaborator

@bwoebi bwoebi commented Mar 6, 2024

Description

Fixes src/ dir copy and avoids the .dll suffix at all costs for windows.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@bwoebi bwoebi requested a review from a team as a code owner March 6, 2024 17:58
@bwoebi bwoebi force-pushed the bob/relative-ext-path branch from 286d343 to 17bbafb Compare March 6, 2024 18:03
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 6, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.91%. Comparing base (f5c4106) to head (17bbafb).
⚠️ Report is 877 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2559   +/-   ##
=========================================
  Coverage     75.91%   75.91%           
  Complexity     2574     2574           
=========================================
  Files           240      240           
  Lines         27029    27029           
  Branches        976      976           
=========================================
  Hits          20519    20519           
  Misses         5990     5990           
  Partials        520      520           
Flag Coverage Δ
appsec-extension 69.13% <ø> (ø)
tracer-extension 78.70% <ø> (ø)
tracer-php 75.08% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5c4106...17bbafb. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bwoebi bwoebi merged commit 1048c3a into master Mar 7, 2024
@bwoebi bwoebi deleted the bob/relative-ext-path branch March 7, 2024 09:45
@github-actions github-actions Bot added this to the 0.99.0 milestone Mar 7, 2024
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