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

[master] Added deprecation warning for Salt's backport of OrderedDict class removal in 3009 #65542

Merged
merged 7 commits into from
Nov 16, 2023

Conversation

dmurphy18
Copy link
Contributor

@dmurphy18 dmurphy18 commented Nov 13, 2023

What does this PR do?

Added deprecation warning for Salt's backport of OrderedDict class removal in 3009

What issues does this PR fix or reference?

Fixes:

New Behavior

Deprecation warning issued about removal of Salt's backport of OrderedDict added in Python 2 days, recommending use of Python 3 OrderedDict.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@dmurphy18 dmurphy18 requested a review from a team as a code owner November 13, 2023 17:12
@dmurphy18 dmurphy18 requested review from dwoz and removed request for a team November 13, 2023 17:12
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title Added deprecation warning for Salt's backport of OrderedDict class removal in 3009 [master] Added deprecation warning for Salt's backport of OrderedDict class removal in 3009 Nov 13, 2023
twangboy
twangboy previously approved these changes Nov 14, 2023
changelog/65542.deprecated.md Outdated Show resolved Hide resolved
salt/utils/odict.py Outdated Show resolved Hide resolved
Co-authored-by: Pedro Algarvio <[email protected]>
salt/utils/odict.py Outdated Show resolved Hide resolved
Co-authored-by: Pedro Algarvio <[email protected]>
s0undt3ch
s0undt3ch previously approved these changes Nov 15, 2023
@s0undt3ch s0undt3ch merged commit 6c7abe7 into saltstack:master Nov 16, 2023
421 checks passed
@dmurphy18 dmurphy18 deleted the odict_deprecat branch March 13, 2024 18:00
Sign up for free to 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.

3 participants