Skip to content

MINOR: [Dev][Archery] Reinstate version constraint on setuptools_scm for comment bot - #40162

Merged
pitrou merged 1 commit into
apache:mainfrom
pitrou:minor-comment-bot
Feb 20, 2024
Merged

MINOR: [Dev][Archery] Reinstate version constraint on setuptools_scm for comment bot#40162
pitrou merged 1 commit into
apache:mainfrom
pitrou:minor-comment-bot

Conversation

@pitrou

Copy link
Copy Markdown
Member

The comment bot depends on an internal setuptools_scm API that was changed in setuptools_scm 8.
We therefore need to reinstate the Archery version constraint that was removed in #40150

See example failure at https://github.com/apache/arrow/actions/runs/7976567301/job/21777437575

…t bot
The comment bot depends on an internal setuptools_scm API that was changed in setuptools_scm 8.
See example failure at https://github.com/apache/arrow/actions/runs/7976567301/job/21777437575
@github-actionsgithub-actionsBot added the awaiting review Awaiting review label Feb 20, 2024
@pitroupitrou changed the title MINOR: [CI] Reinstate version constraint on setuptools_scm for comment botMINOR: [Dev][Archery] Reinstate version constraint on setuptools_scm for comment botFeb 20, 2024
@pitrou
pitrou marked this pull request as ready for review February 20, 2024 17:29
@pitrou

Copy link
Copy Markdown
MemberAuthor

I'll merge as soon as CI is green, so as to (hopefully) make the comment bot working again.

@pitrou
pitrou merged commit 0988933 into apache:mainFeb 20, 2024
@pitroupitrou removed the awaiting review Awaiting review label Feb 20, 2024
@pitrou
pitrou deleted the minor-comment-bot branch February 20, 2024 17:45
@h-vetinari

Copy link
Copy Markdown
Contributor

Thanks for fixing the fallout here!

@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 0988933.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

Sign up for freeto 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

@pitrou@h-vetinari