Skip to content

Conversation

carlopi
Copy link
Contributor

@carlopi carlopi commented Mar 17, 2024

After this only two jobs are directly invoked on creating a tag:

  • Java.yml, that in the "Combine JARs" step actually performs maven upload
  • SwiftRelease.yml, that triggers the Swift release

Both should probably also move out of the push on tags and into a separate (and more testable) workflow, but I'd say for now this is good enough.

Also, adding R.yml to be invokable via workflow_call / workflow_dispatch, and invoke it as part of InvokeCI.

@carlopi carlopi force-pushed the review_ci_on_tags branch from 888dfc8 to 4cac8ad Compare March 17, 2024 21:26
@github-actions github-actions bot marked this pull request as draft March 17, 2024 21:26
@carlopi carlopi marked this pull request as ready for review March 17, 2024 21:26
@github-actions github-actions bot marked this pull request as draft March 18, 2024 10:21
@carlopi carlopi marked this pull request as ready for review March 18, 2024 10:23
@carlopi carlopi requested a review from Mytherin March 18, 2024 10:23
@github-actions github-actions bot marked this pull request as draft March 18, 2024 10:27
@carlopi carlopi marked this pull request as ready for review March 18, 2024 10:28
@Mytherin Mytherin merged commit f63638a into duckdb:main Mar 18, 2024
github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Mar 18, 2024
Merge pull request duckdb/duckdb#11223 from maiadegraaf/verify_vector_flatten
Merge pull request duckdb/duckdb#11225 from Mytherin/juliabumpandfix
Merge pull request duckdb/duckdb#11218 from Mytherin/disablejemalloconarm
Merge pull request duckdb/duckdb#11217 from carlopi/fix_python_and_extensions
Merge pull request duckdb/duckdb#11212 from carlopi/review_ci_on_tags
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Mar 23, 2024
Merge pull request duckdb/duckdb#11218 from Mytherin/disablejemalloconarm
Merge pull request duckdb/duckdb#11217 from carlopi/fix_python_and_extensions
Merge pull request duckdb/duckdb#11212 from carlopi/review_ci_on_tags
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Mar 23, 2024
Merge pull request duckdb/duckdb#11218 from Mytherin/disablejemalloconarm
Merge pull request duckdb/duckdb#11217 from carlopi/fix_python_and_extensions
Merge pull request duckdb/duckdb#11212 from carlopi/review_ci_on_tags
@carlopi carlopi deleted the review_ci_on_tags branch May 7, 2024 08:13
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.

2 participants