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

Merging feature/migration-to-aiohttp, resolves #28 (Work-in-progress), #29

Closed
wants to merge 6 commits into from

Conversation

tpiekarski
Copy link
Owner

Testing async functions is... well horrible.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@tpiekarski
Copy link
Owner Author

tpiekarski commented Jul 14, 2020

Current errors and warnings

Blocking Error at Travis while running Jobs for Python 3.7 and 3.8:

https://travis-ci.org/github/tpiekarski/derl/builds/707355078

$ make run
Testing if derl runs with tests/test-directory (use args="" to pass arguments)
derl tests/test-directory 
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.7.6/bin/derl", line 11, in <module>
    load_entry_point('derl==0.0.post0.dev87+g17bf72d', 'console_scripts', 'derl')()
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/pkg_resources/__init__.py", line 490, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2854, in load_entry_point
    return ep.load()
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2445, in load
    return self.resolve()
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2451, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/derl-0.0.post0.dev87+g17bf72d-py3.7.egg/derl/main.py", line 14, in <module>
    from derl.dispatcher import run_loop
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/derl-0.0.post0.dev87+g17bf72d-py3.7.egg/derl/dispatcher.py", line 7, in <module>
    import asyncio
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/asyncio/__init__.py", line 21, in <module>
    from .base_events import *
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/asyncio/base_events.py", line 296
    future = tasks.async(future, loop=self)
                       ^
SyntaxError: invalid syntax
Makefile:60: recipe for target 'run' failed
make: *** [run] Error 1
The command "make run" exited with 2.

Minor Travis issue

Using /home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages
Finished processing dependencies for derl==0.0.post0.dev87+g17bf72d
writing list of installed files to 'files.log'
/home/travis/build/tpiekarski/derl/.eggs/PyScaffold-3.2.3-py3.7.egg/pyscaffold/contrib/setuptools_scm/git.py:68: UserWarning: "/home/travis/build/tpiekarski/derl" is shallow and may cause errors
  warnings.warn('"%s" is shallow and may cause errors' % (wd.path,))

Local issue with bdist_wheel

Complete output from command /home/todde/dev/derl/.venv-tests/bin/python -u -c [...]
usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
    or: -c --help [cmd1 cmd2 ...]
    or: -c --help-commands
    or: -c cmd --help

error: invalid command 'bdist_wheel'

----------------------------------------
Failed building wheel for pyyaml
Running setup.py clean for pyyaml
Failed to build pyyaml

@tpiekarski tpiekarski changed the title Merging feature/migration-to-aiohttp Merging feature/migration-to-aiohttp, resolves #28 (work-in-progress), Jul 14, 2020
@tpiekarski tpiekarski linked an issue Jul 14, 2020 that may be closed by this pull request
@tpiekarski tpiekarski changed the title Merging feature/migration-to-aiohttp, resolves #28 (work-in-progress), Merging feature/migration-to-aiohttp, resolves #28 (Work-in-progress), Jul 14, 2020
@tpiekarski tpiekarski added the help wanted Extra attention is needed label Jul 14, 2020
@tpiekarski tpiekarski closed this Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate dispatcher from requests to AIOHTTP module
1 participant