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

addDeviceDataAsStream-> write_stream #811

Merged
merged 5 commits into from
Feb 17, 2023
Merged

addDeviceDataAsStream-> write_stream #811

merged 5 commits into from
Feb 17, 2023

Conversation

prjemian
Copy link
Contributor

@prjemian prjemian added this to the 1.6.14 milestone Feb 17, 2023
@prjemian prjemian self-assigned this Feb 17, 2023
@prjemian
Copy link
Contributor Author

The super_linter reports:

2023-02-17 08:15:09 [ERROR]   Found errors in [mypy] linter!
2023-02-17 08:15:09 [ERROR]   Error code: 1. Command output:
------
WARNING: The directory '/github/home/.cache/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you should use sudo's -H flag.
Collecting types-PyYAML
  Downloading types_PyYAML-6.0.12.6-py3-none-any.whl (14 kB)
Installing collected packages: types-PyYAML
Successfully installed types-PyYAML-6.0.12.6

Notice:  A new release of pip available: 22.3.1 -> 23.0
Notice:  To update, run: python -m pip install --upgrade pip
apstools/plans/command_list.py:419: error: Incompatible types in assignment (expression has type "Callable[[Any, Any, Any], Any]", variable has type "None")  [assignment]
Installing missing stub packages:
/venvs/mypy/bin/python -m pip install types-PyYAML


Found 1 error in 1 file (checked 1 source file)
------

As noted in #809 (comment), this message is not related to this issue and it is not useful. Disable the super_linter workflow.

@prjemian prjemian merged commit 183f473 into main Feb 17, 2023
@prjemian prjemian deleted the 797-write_stream branch February 17, 2023 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor and use addDeviceDataAsStream()
1 participant