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

[device/accton] Fix Python error in sfp.py for as7116-54x #4391

Merged
merged 1 commit into from
Apr 9, 2020
Merged

[device/accton] Fix Python error in sfp.py for as7116-54x #4391

merged 1 commit into from
Apr 9, 2020

Conversation

jleveque
Copy link
Contributor

@jleveque jleveque commented Apr 8, 2020 •
edited
Loading

Fix syntax error

@jleveque jleveque requested a review from lguohan April 8, 2020 03:45
@jleveque jleveque self-assigned this Apr 8, 2020
@jleveque
Copy link
Contributor Author

jleveque commented Apr 8, 2020

Retest vsimage please

2 similar comments
@jleveque
Copy link
Contributor Author

jleveque commented Apr 8, 2020

Retest vsimage please

@jleveque
Copy link
Contributor Author

jleveque commented Apr 9, 2020

Retest vsimage please

@lguohan lguohan merged commit 21aff7f into sonic-net:master Apr 9, 2020
@jleveque jleveque deleted the fix_python_warnings_accton_as7116_54x branch April 9, 2020 17:57
abdosi pushed a commit that referenced this pull request Apr 11, 2020
@jleveque @abdosi
[device/accton] Fix Python error for as7116-54x (#4391)
2772cce 
Fix syntax error
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 24, 2020
@jleveque @tiantianlv
[device/accton] Fix Python error for as7116-54x (sonic-net#4391)
b0b912b 
Fix syntax error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan approved these changes

Assignees

@jleveque jleveque

Labels
Bug 🐛 Included in 201911 Branch Platform: Accton Request for 201911 Branch :shipit:
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jleveque @lguohan @abdosi