Skip to content

ci(test-all-versions): this workflow needs install scripts for tests to pass#3161

Merged
maryliag merged 2 commits intoopen-telemetry:mainfrom
trentm:trentm-tav-workflow-needs-install-scripts
Oct 8, 2025
Merged

ci(test-all-versions): this workflow needs install scripts for tests to pass#3161
maryliag merged 2 commits intoopen-telemetry:mainfrom
trentm:trentm-tav-workflow-needs-install-scripts

Conversation

@trentm
Copy link
Copy Markdown
Contributor

@trentm trentm commented Oct 7, 2025

https://github.com/open-telemetry/opentelemetry-js-contrib/actions/workflows/test-all-versions.yml started
failing last week after #3121. We cannot use --ignore-scripts in this case. This is
the equivalent change as was done for test.yml in #3122.

@trentm trentm self-assigned this Oct 7, 2025
@trentm trentm requested a review from a team as a code owner October 7, 2025 21:14
@maryliag maryliag merged commit b6afe88 into open-telemetry:main Oct 8, 2025
17 checks passed
trentm added a commit to trentm/opentelemetry-js-contrib that referenced this pull request Oct 8, 2025
…to pass (open-telemetry#3161)

Co-authored-by: Marylia Gutierrez <marylia.gutierrez@grafana.com>
@trentm trentm deleted the trentm-tav-workflow-needs-install-scripts branch October 8, 2025 22:30
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.

2 participants