Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merge main into v1.1.3 #116

Merged
merged 54 commits into from
Nov 25, 2024
Merged

Merge main into v1.1.3 #116

merged 54 commits into from
Nov 25, 2024

Conversation

samansmink
Copy link
Collaborator

No description provided.

samansmink and others added 30 commits October 15, 2024 11:20
@samansmink
small ci tweaks
8d4a9e2
@samansmink
add skipping option to makefile
5f70495
@samansmink
call test_release explicitly instead of test
5f13e93
@samansmink
use : not =
f564633
@samansmink
skip tests outside docker for now
df506fd
@samansmink
tabs not whitespaces
12c1fc2
@samansmink
fix windows; switch to make configure_ci step instead of custom_toolc…
e4955f0 
…hain_script
@samansmink
allow passing platform as file
3d1c62e
@samansmink
update workflow for upcoming rust template
0d7e751
@samansmink
cleaner error handling in metadata script
3930e25
@samansmink
fix disabling test outside docker
fbcf3da
@samansmink
cleaning comments for skipping
74f0e97
@samansmink
switch to matrix distro script in ci
6ec869b
@samansmink
Merge pull request #98 from samansmink/minor-ci-tweaks
bb3809d 
Prepare ci tools for extensions without DuckDB submodule
@samansmink
add platform/version detection script
795955c
@samansmink
add makefiles for C extensions
27facd3
@samansmink
Merge pull request #101 from samansmink/minor-scipt-additions
3e987be 
Add new Makefiles for experimental C API based extensions
@samansmink
bump download artifact
df8d46f
@samansmink
also bump upload
21d3a39
@samansmink
Merge pull request #102 from samansmink/bump-download-artifact
5cd6f79 
Bump download-artifact action
@samansmink
simplify matrix generation step of build
ab001a8
@samansmink
simplify matrix creation step for deploy
74cca85
@joewalnes
Fix hardcoded extension_name
f164d86
@samansmink
Merge pull request #104 from joewalnes/fix-extension-name
eb07f23 
Fix hardcoded extension_name
@samansmink
Merge pull request #103 from samansmink/refactor-matrix-generation
0315837 
Refactor matrix generation
@carlopi
formatting
cd15603
@carlopi
Add EXTENSION_FILENAME_NO_METADATA concept
a1a430d
@carlopi
Add wasm detection, and handling both in Capi and rust
9b64725
@carlopi
Workaround for crate-type
1f7a0fe
@carlopi
emscripten invokation: skip verbose
78eb998
carlopi and others added 24 commits November 13, 2024 01:44
@carlopi
Merge pull request #105 from carlopi/fixes_wasm
5bdbe4d 
Fixes to enable building for Wasm targets
@carlopi
Move Wasm targets instructions into extension-ci-tools
ce31455
@samansmink
Merge pull request #106 from carlopi/more_fixes_wasm
d3a5766 
More fixes for Wasm builds, and move configure one layer up
@samansmink
add c makefile
ca1ea12
@samansmink
two small tweaks
f801a7a
@samansmink
rename
4364677
@samansmink
Merge pull request #108 from samansmink/add-c-makefile-2
871db4d 
add c makefile
@samansmink
add new templates to CI
37750a4
@samansmink
disable non-working archs
92969bb
@samansmink
trigger ci
e8b396a
@samansmink
disable rust threads ci run
902ed05
@carlopi
Support also Wasm targets
a4c1050
@carlopi
Merge pull request #110 from carlopi/wasm_support_c
9fd9123 
[C/C++ CAPI extensions] Support also Wasm targets
@carlopi
Fixup invocation
3d16a4d
@carlopi
Actually fix properly for ninja
83c4013
@samansmink
Merge pull request #109 from samansmink/add-new-templates-to-ci
512b17d 
add new templates to CI
@carlopi
Fixup cmake --build invocation for Wasm targets
271ebe1 
Problem is that emscriten wrapped cmake and naked cmake take slighlty
different ways to actually build, so we need to abstract this away
@carlopi
vcpkg: Add handling for Wasm targets in c_api
0c1e35e
@carlopi
Move from wasm_coi to wasm_threads
f950015
@samansmink
Merge pull request #113 from carlopi/wasm_support_c
afbb4dc 
Wasm support for extension-template-c
@carlopi
Enable Wasm for templates and remove unnecessary toolchains
78529d2
@samansmink
Merge pull request #114 from carlopi/enable_wasm_templates
916d4ef 
Enable Wasm for templates and remove unnecessary toolchains
@samansmink
remove pkg-config brew install
b712ab8
@samansmink
Merge pull request #115 from samansmink/remove-pkg-config
46a8dd6 
remove pkg-config brew install
@carlopi carlopi self-requested a review November 25, 2024 09:45
@Dtenwolde Dtenwolde mentioned this pull request Nov 25, 2024
Merged
carlopi
carlopi approved these changes Nov 25, 2024
View reviewed changes
Copy link
Collaborator

@carlopi carlopi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@samansmink samansmink merged commit 5121955 into v1.1.3 Nov 25, 2024
80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carlopi carlopi carlopi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@samansmink @carlopi @joewalnes