Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix azure service bus isinstance checks when None #2053

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

jasonwbarnett
Copy link
Contributor

@jasonwbarnett jasonwbarnett commented Jul 9, 2024

Fixes #2045 and #1947

@jasonwbarnett jasonwbarnett force-pushed the jwb/fix-missing-dep branch from 8edd7d1 to 1867154 Compare July 9, 2024 18:20
@jasonwbarnett jasonwbarnett changed the title add azure-identity to azure servicebus requirements fix azure service bus isinstance checks when None Jul 9, 2024
Copy link
Member

@Nusnus Nusnus 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 quick fix!

Just fix the lint issue and we can merge 🚀

pyupgrade................................................................Passed
autoflake................................................................Passed
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

kombu/transport/azureservicebus.py:144:80: E501 line too long (109 > 79 characters)
kombu/transport/azureservicebus.py:145:80: E501 line too long (111 > 79 characters)
kombu/transport/azureservicebus.py:146:16: E111 indentation is not a multiple of 4
kombu/transport/azureservicebus.py:146:16: E117 over-indented

Strip unnecessary `# noqa`s..............................................Passed
check for merge conflicts................................................Passed
check toml...........................................(no files to check)Skipped
check yaml...............................................................Passed
mixed line ending........................................................Passed
isort....................................................................Passed

Copy link
Member

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

please fix the lint errors

@jasonwbarnett jasonwbarnett force-pushed the jwb/fix-missing-dep branch 2 times, most recently from d187a05 to 06b8c10 Compare July 10, 2024 21:12
@jasonwbarnett
Copy link
Contributor Author

@Nusnus @auvipy done

Copy link
Member

@Nusnus Nusnus left a comment

Choose a reason for hiding this comment

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

Thanks!

@Nusnus Nusnus merged commit 52d8b7f into celery:main Jul 10, 2024
16 checks passed
@jasonwbarnett jasonwbarnett deleted the jwb/fix-missing-dep branch July 17, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure servicebus transport has hidden dependency on azure-identity package
3 participants