refactor!(kernels): convert split_edge routines to use transactions

[imrn99]

Description

Scope: kernels

Type of change: refactor

Content description:

• rename split_edge_no_alloc & splitn_edge_no_alloc to split_edge_transac & splitn_edge_transac
• update API of both methods to accept transactions
• update SplitEdgeError to have a failed transaction variant
• update common internal routine to use a transaction

Additional information

• Breaking change

Necessary follow-up

merge this back into #269 and finish writing the new benchmark.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 84.41558% with 36 lines in your changes missing coverage. Please review.

Project coverage is 85.09%. Comparing base (6e38028) to head (b36ea20).

Files with missing lines	Patch %	Lines
honeycomb-kernels/src/splits/tests.rs	73.49%	22 Missing ⚠️
honeycomb-kernels/src/splits/edge_multiple.rs	95.38%	3 Missing ⚠️
honeycomb-kernels/src/splits/edge_single.rs	94.82%	3 Missing ⚠️
honeycomb-kernels/src/splits/mod.rs	0.00%	3 Missing ⚠️
...-kernels/src/grisubal/routines/insert_intersecs.rs	81.81%	2 Missing ⚠️
...-kernels/src/grisubal/routines/insert_new_edges.rs	80.00%	2 Missing ⚠️
honeycomb-kernels/src/grisubal/mod.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   85.14%   85.09%   -0.05%     
==========================================
  Files          62       63       +1     
  Lines        8924     9024     +100     
==========================================
+ Hits         7598     7679      +81     
- Misses       1326     1345      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cedricchevalier19]

LGTM