Skip to content

test: DynamicVersion#21

Merged
LecrisUT merged 7 commits intomainfrom
test/DynamicVersion
Jan 3, 2024
Merged

test: DynamicVersion#21
LecrisUT merged 7 commits intomainfrom
test/DynamicVersion

Conversation

@LecrisUT
Copy link
Copy Markdown
Owner

@LecrisUT LecrisUT commented Jan 3, 2024

Fixed a few bugs with DynamicVersion and expanded the tests

Signed-off-by: Cristian Le <[email protected]>
Signed-off-by: Cristian Le <[email protected]>
Signed-off-by: Cristian Le <[email protected]>
- Files were not copied which affects `CMAKE_CONFIGURE_DEPENDS`
- Git archive saved the wrong values from regex
- Wrong json escape in fallback

Signed-off-by: Cristian Le <[email protected]>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 3, 2024

Test Results

 3 files  ±0   3 suites  ±0   29s ⏱️ +20s
 5 tests ±0   5 ✅ ±0  0 💤 ±0  0 ❌ ±0 
15 runs  ±0  15 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 36342c2. ± Comparison against base commit 69e2f44.

♻️ This comment has been updated with latest results.

Signed-off-by: Cristian Le <[email protected]>
@LecrisUT LecrisUT force-pushed the test/DynamicVersion branch from 4f558d0 to 7361ccd Compare January 3, 2024 18:21
@LecrisUT LecrisUT mentioned this pull request Jan 3, 2024
4 tasks
@LecrisUT LecrisUT force-pushed the test/DynamicVersion branch 3 times, most recently from 89be881 to fafe142 Compare January 3, 2024 19:09
Signed-off-by: Cristian Le <[email protected]>
@LecrisUT LecrisUT force-pushed the test/DynamicVersion branch from fafe142 to 36342c2 Compare January 3, 2024 19:15
@LecrisUT LecrisUT merged commit 958ed05 into main Jan 3, 2024
@LecrisUT LecrisUT deleted the test/DynamicVersion branch January 5, 2024 09:38
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.

1 participant