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

Summary: add ruijie b6510 device #4419

Closed
wants to merge 1 commit into from
Closed

Summary: add ruijie b6510 device #4419

wants to merge 1 commit into from

Conversation

tim-rj
Copy link
Contributor

@tim-rj tim-rj commented Apr 13, 2020

- What I did
add support of ruijie b6510 device
- How I did it
add ruijie b6510 platform and device
- How to verify it
compile SONiC bin and run in ruijie b6510 device
- Description for the changelog
add ruijie b6510 device

- A picture of a cute animal (not mandatory but encouraged)

@lgtm-com
Copy link

lgtm-com bot commented Apr 13, 2020

This pull request introduces 68 alerts when merging 5dd89d4 into b6ae12b - view on LGTM.com

new alerts:

  • 23 for Unused import
  • 10 for Variable defined multiple times
  • 6 for Unreachable code
  • 6 for 'import *' may pollute namespace
  • 5 for Unnecessary pass
  • 5 for Testing equality to None
  • 5 for Unused local variable
  • 3 for Redundant comparison
  • 2 for Except block handles 'BaseException'
  • 1 for Syntax error
  • 1 for Non-iterable used in for loop
  • 1 for Use of 'global' at module level

@lguohan
Copy link
Collaborator

lguohan commented Apr 15, 2020

for new platforms, please submit to master branch first.

@lguohan
Copy link
Collaborator

lguohan commented May 21, 2020

new device needs to go to master first.

@lguohan lguohan closed this May 21, 2020
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