Skip to content

Conversation

@ezhang6811
Copy link
Contributor

@ezhang6811 ezhang6811 commented Sep 24, 2025

Issue #, if available:

Description of changes:
Add a workflow nightly-build.yml with following actions:

Example PR: #287

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ezhang6811 ezhang6811 requested a review from a team as a code owner September 24, 2025 01:54
@ezhang6811 ezhang6811 added the skip changelog doesn't need a CHANGELOG entry label Sep 24, 2025
Copy link
Contributor

@thpierce thpierce left a comment

Choose a reason for hiding this comment

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

Only reviewed .github/workflows

Comment on lines +8 to +10
push:
branches:
- zhaez/nightly-build
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove

secrets: inherit
permissions:
id-token: write
contents: write
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not read?

Comment on lines +51 to +55
- name: Check for breaking changes
id: breaking_changes
env:
OTEL_DOTNET_CORE_VERSION: ${{ steps.get_versions.outputs.otel_dotnet_core_version }}
run: python scripts/find_breaking_changes.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not check OTEL_DOTNET_INSTRUMENTATION_VERSION?

echo "has_changes=true" >> $GITHUB_OUTPUT

git add .
git commit -m "chore: update OpenTelemetry dependencies to ${{ steps.get_versions.outputs.otel_dotnet_core_version }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not OTEL_DOTNET_INSTRUMENTATION_VERSION?

else
echo "Creating new PR..."
gh pr create \
--title "Nightly dependency update: OpenTelemetry ${{ needs.update-dependencies.outputs.otel_dotnet_core_version }}" \
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not OTEL_DOTNET_INSTRUMENTATION_VERSION?

Comment on lines +126 to +127
- [OpenTelemetry .NET Core](https://github.com/open-telemetry/opentelemetry-dotnet/releases/tag/v${{ needs.update-dependencies.outputs.otel_dotnet_core_version }}): ${{ needs.update-dependencies.outputs.otel_dotnet_core_version }}
- OpenTelemetry .NET Contrib packages: Updated to latest available versions
Copy link
Contributor

Choose a reason for hiding this comment

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

OTEL_DOTNET_INSTRUMENTATION_VERSION?

Why latest available versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog doesn't need a CHANGELOG entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants