Skip to content

Conversation

@trask
Copy link
Member

@trask trask commented Oct 18, 2024

I think it's failing because we aren't using --refresh-dependencies in the CI japicmp check, but I'm reluctant to add that flag b/c then it would slow down all builds

I think it should clear itself up in <24 hours, maybe we can think of something better...

@trask trask requested a review from a team as a code owner October 18, 2024 00:51
@trask trask enabled auto-merge (squash) October 18, 2024 01:17
Comment on lines -168 to +169
echo $(git diff --cached --name-only)
echo $(git diff --cached)
git diff --cached --name-only
git diff --cached
Copy link
Member Author

Choose a reason for hiding this comment

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

this improved the output, so I'd suggest keeping the change

@laurit
Copy link
Contributor

laurit commented Oct 18, 2024

I think it's failing because we aren't using --refresh-dependencies in the CI japicmp check, but I'm reluctant to add that flag b/c then it would slow down all builds

I think it should clear itself up in <24 hours, maybe we can think of something better...

I disabled caching for the latest release check. Maybe with this we don't need --refresh-dependencies at all. Idk whether this will have any unexpected side effects.

@trask trask merged commit 67cb5f1 into open-telemetry:main Oct 18, 2024
55 of 56 checks passed
@trask trask deleted the post-release branch October 18, 2024 07:15
@trask
Copy link
Member Author

trask commented Oct 18, 2024

I disabled caching for the latest release check. Maybe with this we don't need --refresh-dependencies at all. Idk whether this will have any unexpected side effects.

nice!

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