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

Streamline scripts in Firmware/tools. #51

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

Conversation

SamwelOpiyo
Copy link

New:

  • Add a requirements.txt to specify script dependencies.
  • Add import of pexpect

Changes

  • Correct import of fdpexpect.

New:

* Add a requirements.txt to specify script dependencies.
* Add import of pexpect and correct import of fdpexpect.
@kaklik
Copy link
Contributor

kaklik commented Nov 6, 2022

That seems to also resolve the: #47 please merge. Thanks.

kaklik added a commit to ThunderFly-aerospace/SiK that referenced this pull request Nov 6, 2022
@SamwelOpiyo
Copy link
Author

@kaklik, looks like I do not have write access and therefore only those with write access can merge this pull request.

@kaklik
Copy link
Contributor

kaklik commented Nov 7, 2022

@kaklik, looks like I do not have write access and therefore only those with write access can merge this pull request.

Yes, I have the same problem. I do not know anybody except @tridge who has maintainer rights to this repository. At least, I merged it to our ThunderFly fork, which I try to maintain more regularly.

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