Skip to content

Conversation

hannes
Copy link
Member

@hannes hannes commented Feb 27, 2025

I'm not 100% on the conditional

      if: github.repository == 'duckdb/duckdb' && github.event_name != 'pull_request'

perhaps someone else knows more or we just need a few attempts

@carlopi
Copy link
Contributor

carlopi commented Feb 27, 2025

Looks right! Should we give this a go for v1.2.1 (if so, needs retargeting to v1.2-histrionicus)?

Either way proper test is after merge on the first invocation of Windows.yml (that can also be triggered on purpose, just to reduce latency)

@hannes hannes changed the base branch from main to v1.2-histrionicus February 27, 2025 09:15
@hannes hannes changed the base branch from v1.2-histrionicus to main February 27, 2025 09:16
@hannes hannes changed the base branch from main to v1.2-histrionicus February 27, 2025 10:15
@duckdb-draftbot duckdb-draftbot marked this pull request as draft February 27, 2025 10:15
@hannes
Copy link
Member Author

hannes commented Feb 27, 2025

re-targeted to 1.2 should be good to go, we'll see after merge

@hannes hannes marked this pull request as ready for review February 27, 2025 10:15
@Mytherin Mytherin merged commit 114cdef into duckdb:v1.2-histrionicus Feb 27, 2025
9 of 10 checks passed
@Mytherin
Copy link
Collaborator

Awesome, thanks!

@hannes
Copy link
Member Author

hannes commented Feb 27, 2025

Looks like it worked https://github.com/duckdb/duckdb/actions/runs/13568920626/job/37928568179
So I think no further action is required.

krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Mar 7, 2025
CSV small code Improvements + initialising boolean variable. (duckdb/duckdb#16454)
Adding windows code signing using azure (duckdb/duckdb#16444)
Bump delta to working commit (duckdb/duckdb#16442)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants