Skip to content

Conversation

@bsiegel
Copy link
Contributor

@bsiegel bsiegel commented May 25, 2021

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

@BrewTestBot BrewTestBot added the python Python use is a significant feature of the PR or issue label May 25, 2021
@bsiegel bsiegel marked this pull request as ready for review May 25, 2021 20:18
Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @bsiegel. However, I believe azure-cli isn't compatible with Python3.9 yet. (e.g. Azure/azure-cli#15580)

azure-cli's automated release workflow will just switch this back to Python3.8 at the next version bump. See:

https://github.com/Azure/azure-cli/blob/dev/scripts/release/homebrew/docker/formula_template.txt
Azure/azure-cli#15482
#62253
#62785

@bsiegel
Copy link
Contributor Author

bsiegel commented May 25, 2021

Ah, okay. The tracking issue for Python 3.9 support (Azure/azure-cli#16611) was closed and the release notes for 2.24.0 mention Python 3.9 support so I believe the tool itself is now ready for Python 3.9. However as you point out the automation seems like it will just overwrite this change, so it sounds like the right thing to do will be to open a PR to update the automation in the azure-cli repo.

@bsiegel
Copy link
Contributor Author

bsiegel commented May 25, 2021

I've opened Azure/azure-cli#18224 to update the formula template

@carlocab
Copy link
Member

Great, this should be good then once Azure/azure-cli#18224 is merged. Thanks for working on that.

@carlocab
Copy link
Member

carlocab commented Jun 1, 2021

This needs rebasing after #78459. (With git rebase and not git merge, please. Apologies if that's obvious--I've seen others use git merge to rebase before.)

@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale No recent activity label Jun 23, 2021
@bsiegel
Copy link
Contributor Author

bsiegel commented Jun 23, 2021

Still pending merge of the blocking PR in the azure-cli repo. Looks like it -may- be scheduled for completion this month.

@carlocab carlocab added in progress Stale bot should stay away upstream issue An upstream issue report is needed and removed stale No recent activity labels Jun 23, 2021
@cho-m cho-m mentioned this pull request Aug 11, 2021
@jiasli
Copy link

jiasli commented Aug 13, 2021

We have merged Azure/azure-cli#18224. Please go ahead and merge this PR. Thanks!!

carlocab
carlocab previously approved these changes Aug 13, 2021
@BrewTestBot
Copy link
Contributor

:shipit: @carlocab has triggered a merge.

@carlocab
Copy link
Member

Thanks, @bsiegel, @jiasli!

@BrewTestBot
Copy link
Contributor

⚠️ @carlocab bottle publish failed.

@BrewTestBot BrewTestBot dismissed carlocab’s stale review August 13, 2021 14:00

bottle publish failed

@carlocab
Copy link
Member

Oops, this doesn't work. Let me try something else.

@carlocab
Copy link
Member

Cherry-picked the changes here onto #83088.

@carlocab carlocab closed this Aug 13, 2021
@github-actions github-actions bot added the outdated PR was locked due to age label Sep 13, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

in progress Stale bot should stay away outdated PR was locked due to age python Python use is a significant feature of the PR or issue upstream issue An upstream issue report is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants