Skip to content
New issue

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

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

Already on GitHub? # to your account

Update UTs regarding deprecations #387

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Dec 5, 2023

With latest changes there were some deprecations, most notably the default ctor of DRSS. Due that the build started emitting quite some warning messages during build.

This is now lowered to 2 warnings for whole build. There is no issue for this, as these changes are all happening in UTs.

With latest changes there were some deprecations,
most notably the default ctor of DRSS. Due that
the build started emitting quite some warning
messages during build.

This is not lowered to 2 warnings for whole build.
@cstamas cstamas added this to the 2.0.0 milestone Dec 5, 2023
@cstamas cstamas self-assigned this Dec 5, 2023
@cstamas cstamas changed the title Just a general code health improvement Update UTs regarding deprecations Dec 5, 2023
@cstamas cstamas merged commit 45c5bdf into apache:master Dec 5, 2023
@cstamas cstamas deleted the lessen-build-warnings branch December 5, 2023 18:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant