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

Bump binary package version #19190

Merged
merged 8 commits into from
Dec 5, 2024
Merged

Bump binary package version #19190

merged 8 commits into from
Dec 5, 2024

Conversation

HadhemiDD
Copy link
Contributor

What does this PR do?

Bump the binary package version to fix a warning that emerged with python 3.12.
The PR link with the fix: ofek/binary#11

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Copy link

codecov bot commented Dec 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.36%. Comparing base (59e8e20) to head (1a8613e).
Report is 1 commits behind head on master.

Additional details and impacted files
Flag Coverage Δ
activemq ?
cassandra ?
datadog_checks_base 89.59% <ø> (+0.29%) ⬆️
hive ?
hivemq ?
hudi ?
ignite ?
jboss_wildfly ?
kafka ?
presto ?
solr ?

Flags with carried forward coverage won't be shown. Click here to find out more.

@@ -2,7 +2,7 @@ aerospike==7.1.1; sys_platform != 'win32' and sys_platform != 'darwin'
aws-requests-auth==0.4.3
azure-identity==1.17.1
beautifulsoup4==4.12.3
binary==1.0.0
binary==1.0.1; python_version >= '3.12'
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we really need the conditional here? Do we expect this to be backported to py3.11 agents? If so shouldn't there be a line that specify what version to use if the python is not 3.12 and above?

binary==1.0.0; python_version < '3.12'

steveny91
steveny91 previously approved these changes Dec 5, 2024
Copy link
Contributor

@iliakur iliakur left a comment

Choose a reason for hiding this comment

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

changelog?

@HadhemiDD HadhemiDD removed this pull request from the merge queue due to a manual request Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

The changelog type changed or removed was used in this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the fixed or added type instead.

Copy link

github-actions bot commented Dec 5, 2024

The changelog type changed or removed was used in this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the fixed or added type instead.

@HadhemiDD HadhemiDD merged commit e19e029 into master Dec 5, 2024
41 checks passed
@HadhemiDD HadhemiDD deleted the hadhemi/bump-binary-package branch December 5, 2024 19:30
datadog-agent-integrations-bot bot pushed a commit that referenced this pull request Dec 5, 2024
* bump binary package version

* add python version condition

* remove python version condition

* changelo

* changelog modified

* changelog modified

* changelog modified

(cherry picked from commit e19e029)
datadog-agent-integrations-bot bot pushed a commit that referenced this pull request Dec 5, 2024
* bump binary package version

* add python version condition

* remove python version condition

* changelo

* changelog modified

* changelog modified

* changelog modified

(cherry picked from commit e19e029)
HadhemiDD added a commit that referenced this pull request Dec 5, 2024
* bump binary package version

* add python version condition

* remove python version condition

* changelo

* changelog modified

* changelog modified

* changelog modified

(cherry picked from commit e19e029)

Co-authored-by: HadhemiDD <[email protected]>
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.

3 participants