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

devtool: remove stale import. #61

Open
wants to merge 1 commit into
base: scarthgap
Choose a base branch
from
Open

Conversation

peteut
Copy link

@peteut peteut commented Jun 18, 2024

yaml is imported but not used. This may cause issues as yaml is not a standard library module.

@johntoomey
Copy link
Contributor

argparse and sys also appear to be unused. glob should be moved to the top of the list.

@peteut
Copy link
Author

peteut commented Jun 18, 2024

@johntoomey will update accordingly.

* `yaml` is imported but not used. This may cause issues as `yaml`
  is not a standard library module.
* Remove trailing newlines.
@grsandeep85
Copy link
Contributor

@peteut thanks for the patch I will cherry pick and run through the CI.

mhatle pushed a commit that referenced this pull request Jul 15, 2024
Based on github Pull Request #61.

1. `yaml` is imported but not used. This may cause issues as `yaml`
  is not a standard library module.
2. Remove trailing newlines.

Signed-off-by: Alain Péteut <[email protected]>

Update commit message and added PR number in commit message.

Also update import list order.

Signed-off-by: Sandeep Gundlupet Raju <[email protected]>
Signed-off-by: Mark Hatle <[email protected]>
msreeram-xilinx pushed a commit that referenced this pull request Oct 30, 2024
Based on github Pull Request #61.

1. `yaml` is imported but not used. This may cause issues as `yaml`
  is not a standard library module.
2. Remove trailing newlines.

Signed-off-by: Alain Péteut <[email protected]>

Update commit message and added PR number in commit message.

Also update import list order.

Signed-off-by: Sandeep Gundlupet Raju <[email protected]>
Signed-off-by: Mark Hatle <[email protected]>
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.

3 participants