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

Python3: replaced ConfigParser module #4005

Merged
merged 1 commit into from
Oct 29, 2024
Merged

Python3: replaced ConfigParser module #4005

merged 1 commit into from
Oct 29, 2024

Conversation

bskjois
Copy link
Contributor

@bskjois bskjois commented Sep 23, 2024

Signed-off-by: Kowshik Jois B S [email protected]

@bskjois
Copy link
Contributor Author

bskjois commented Oct 2, 2024

@cliping @chunfuwen @dzhengfy @chloerh @Yingshun @xiaodwan
May I request one of you to review and merge this simple PR!

@bskjois
Copy link
Contributor Author

bskjois commented Oct 14, 2024

@cliping @chunfuwen @dzhengfy @chloerh @Yingshun @xiaodwan
Could one of you take a look at this PR?
Its a very simple one and its been more than 3 weeks. I am just replacing a derecated function.
Thanks in Advance!

@chloerh chloerh merged commit 452d4c2 into avocado-framework:master Oct 29, 2024
50 checks passed
@bskjois bskjois deleted the ConfigParser branch October 29, 2024 13:45
@bskjois
Copy link
Contributor Author

bskjois commented Oct 29, 2024

@xiaodwan @chloerh

Thank you so much for the quick review and merger of this PR.
As I mentioned in my comments, I have removed py2 support for ConfigParser in all the files.

Please take a look: #4020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xiaodwan xiaodwan xiaodwan approved these changes

@chloerh chloerh chloerh 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.
3 participants
@bskjois @chloerh @xiaodwan