Convert CDT.Core to deterministic integer arithmetic for Unity compatibility#17
Closed
PeterOlsted wants to merge 1 commit into
Closed
Annotations
11 errors and 11 warnings
|
Add coverage PR comment
Resource not accessible by integration - https://docs.github.com/rest/issues/comments#create-an-issue-comment
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L241
The name 'PredicatesExact' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L225
The name 'PredicatesAdaptive' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L206
The name 'PredicatesExact' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L191
The name 'PredicatesAdaptive' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L176
The name 'PredicatesExact' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L161
The name 'PredicatesAdaptive' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L142
The name 'PredicatesExact' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L127
The name 'PredicatesAdaptive' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L112
The name 'PredicatesExact' does not exist in the current context
|
|
Build:
benchmark/CDT.Benchmarks/PredicatesBenchmarks.cs#L97
The name 'PredicatesAdaptive' does not exist in the current context
|
|
Complete job
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-dotnet@v4, marocchino/sticky-pull-request-comment@v2. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'v1' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'p' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/Predicates/Int256.cs#L261
Ambiguous reference in cref attribute: 'System.Diagnostics.Debug.Assert'. Assuming 'System.Diagnostics.Debug.Assert(bool)', but could have also matched other overloads including 'System.Diagnostics.Debug.Assert(bool, string?)'.
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'v2' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'v1' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'p' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/Predicates/Int256.cs#L261
Ambiguous reference in cref attribute: 'System.Diagnostics.Debug.Assert'. Assuming 'System.Diagnostics.Debug.Assert(bool)', but could have also matched other overloads including 'System.Diagnostics.Debug.Assert(bool, string?)'.
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'v2' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'v1' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
|
Build:
src/CDT.Core/CdtUtils.cs#L334
Parameter 'p' has no matching param tag in the XML comment for 'CdtUtils.LocatePointLine(V2i, V2i, V2i, long)' (but other parameters do)
|
background
wait
wait-all
cancel
parallel
Loading