Skip to content

Remove backslash from f-string (Patch for Issue: #247) #248

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

Merged
merged 2 commits into from
Jan 11, 2025

Conversation

Sakthivel-Ramasamy
Copy link

Remove backslash from f-string to maintain backward compatibility with Python 3.10 and 3.11 versions.

Also, corrected little (very minor non-impacting) cosmetic issues.

Patch for Issue: #247

Remove backslash from f-string to maintain backward compatibility with Python 3.10 and 3.11 versions.

Also, corrected little (very minor non-impacting) cosmetic issues.
@Joezanini
Copy link
Collaborator

Hello Sakthivel, unfortunately, when I am running the test suite on this PR, it is failing a lot of tests. Can you work with the test suite? let me know the output?

"make format" runs
@Sakthivel-Ramasamy
Copy link
Author

Hi Joe,

The test suites have passed now.
image

Thanks,
Sakthi

@Joezanini Joezanini merged commit 99a570d into WebexCommunity:main Jan 11, 2025
@Joezanini
Copy link
Collaborator

Passed on my end as well. Thanks!!

Sakthivel-Ramasamy pushed a commit to Sakthivel-Ramasamy/webex_bot_forked that referenced this pull request Jan 11, 2025
Updated webexpythonsdk library version to 2.0.3 which has the fixed the Python 3.10 and 3.11 version backward compatibility issue, introduced by webexpythonsdk 2.0.2 release (WebexCommunity/WebexPythonSDK#248).
fbradyirl added a commit to fbradyirl/webex_bot that referenced this pull request Apr 24, 2025
* Updated webexpythonsdk library version

Updated webexpythonsdk library version to 2.0.3 which has the fixed the Python 3.10 and 3.11 version backward compatibility issue, introduced by webexpythonsdk 2.0.2 release (WebexCommunity/WebexPythonSDK#248).

* Corrected webexpythonsdk version requirement

Pointed webex_bot to always use webexpythonsdk 2.0.3 or higher

* Update setup.py

Point "setup.py" to exact latest webexpythonsdk version

Co-authored-by: FB <fbradyirl@users.noreply.github.com>

* Update requirements_dev.txt

Point "requirements_dev.txt" to exact latest webexpythonsdk version

Co-authored-by: FB <fbradyirl@users.noreply.github.com>

* Update README.md

Updated README.md file for instructions on using new webex_bot 0.6.0 version

* Update references to Python version

Updated references to Python version

* Updated webexpythonsdk to latest version

Updated webexpythonsdk to latest version 2.0.4

---------

Co-authored-by: Sakthivel Ramasamy (sakthram) <sakthram@cisco.com>
Co-authored-by: FB <fbradyirl@users.noreply.github.com>
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.

2 participants