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.

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

feat(dev/release): add C# post-release script #1595

Conversation

Julian-Brandrick
Copy link
Contributor

post-07-csharp.sh: Uploads Arrow-ADBC NuGet packages to the NuGet.org package manager. Based on the same script from the Arrow repo: dev/release/post-06-csharp.sh
Bumped the indices for other post-release scripts based on the ordering in the Arrow repo.

I packed the Arrow-ADBC csharp folder to get the list of packages to upload and excluded the Test packages:
image

Copy link

github-actions bot commented Mar 7, 2024

⚠️ Please follow the Conventional Commits format in CONTRIBUTING.md for PR titles.

@github-actions github-actions bot added this to the ADBC Libraries 0.11.0 milestone Mar 7, 2024
@lidavidm lidavidm changed the title Adding csharp post-release Arrow-ADBC script and bumping the indices of other scripts feat(dev/release): add C# post-release script Mar 7, 2024
@lidavidm
Copy link
Member

lidavidm commented Mar 7, 2024

Could I trouble you to at least update the filenames in the docs, too? https://github.com/apache/arrow-adbc/blob/main/docs/source/development/releasing.rst

@Julian-Brandrick
Copy link
Contributor Author

Could I trouble you to at least update the filenames in the docs, too? https://github.com/apache/arrow-adbc/blob/main/docs/source/development/releasing.rst

Working on that now David and thank you for the title correction!

base_names+=(Apache.Arrow.Adbc.Drivers.BigQuery.${version})
base_names+=(Apache.Arrow.Adbc.Drivers.FlightSql.${version})
base_names+=(Apache.Arrow.Adbc.Drivers.Interop.Snowflake.${version})
for base_name in ${base_names[@]}; do
Copy link
Member

Choose a reason for hiding this comment

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

In general, we should use "${array[@]}" form:

Suggested change
for base_name in ${base_names[@]}; do
for base_name in "${base_names[@]}"; do

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Applied suggestion

@Julian-Brandrick Julian-Brandrick marked this pull request as ready for review March 15, 2024 23:56
base_names+=(Apache.Arrow.Adbc.Drivers.Interop.Snowflake.${version})
for base_name in "${base_names[@]}"; do
dotnet nuget push \
${base_name}.nupkg \
Copy link
Member

Choose a reason for hiding this comment

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

Where do these packages come from?

Copy link
Member

Choose a reason for hiding this comment

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

Ok, in #1636 I'm adding C# to packaging.yml like all the other builds. Then this script can pull the artifacts from the GitHub release as the other scripts do.

@lidavidm lidavidm merged commit a9ed4c3 into apache:main Mar 18, 2024
19 checks passed
@Julian-Brandrick Julian-Brandrick deleted the dev/v-jbrandrick/CreatingNuGetPostReleaseScript branch May 15, 2024 23:20
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.

3 participants