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.

Sign up for GitHub

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

Jump to bottom

[SDS-435] OrderedDict not needed #115

Merged
merged 1 commit into from
Sep 3, 2021
Merged

[SDS-435] OrderedDict not needed #115

merged 1 commit into from
Sep 3, 2021

Conversation

QFer
Copy link
Contributor

@QFer QFer commented Sep 2, 2021

  • Replaced OrderedDict with dict, reformatting changed code

@QFer QFer mentioned this pull request Sep 2, 2021
Closed
@coveralls
Copy link
Collaborator

Coverage Status

Coverage remained the same at 100.0% when pulling b2efa87 on feature/SDS-NNN/ordereddict into de217f6 on dev.

@QFer QFer changed the title [SDS-NNN] OrderedDict not needed [SDS-435] OrderedDict not needed Sep 2, 2021
@QFer QFer requested a review from peendebak September 2, 2021 17:06
@QFer QFer merged commit 7b32ecb into dev Sep 3, 2021
@QFer QFer deleted the feature/SDS-NNN/ordereddict branch September 3, 2021 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peendebak peendebak peendebak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@QFer @coveralls @peendebak