profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/webknjaz/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Sviatoslav Sydorenko webknjaz @Ansible Core/DevTools @RedHatOfficial Brovary/Kyiv, UA ↝ Brno, CZ https://webknjaz.me 🇺🇦💭🛫✈🛬🛂🛃🛄🏙🚡⛴🏰🏔 * 🐍 3 enthusiast * Gentoo 🐧 jedi * @ansible Core Dev @ Red 🎩 * @aio-libs CI guy * maintainer @ @CherryPy * @open-craft-guild

ansible-community/molecule 3188

Molecule aids in the development and testing of Ansible roles

ansible-community/ansible-lint 2577

Best practices checker for Ansible

aio-libs/async-lru 317

Simple lru cache for asyncio

aio-libs/multidict 257

The multidict implementation

ansible/ansibullbot 190

Bot for management of Ansible issues and PRs on GitHub.

ansible/ansible-lint-action 174

[NOT OFFICIALLY SUPPORTED BY RED HAT BUT BY VOLUNTEERS] | GitHub Action for running ansible-lint as part of your workflows! [ https://github.com/marketplace/actions/ansible-lint ] | PLEASE REPORT `ansible-lint` BUGS TO https://github.com/ansible-community/ansible-lint

aio-libs/aioelasticsearch 128

aioelasticsearch-py wrapper for asyncio

ansible/vscode-ansible 110

vscode/vscodium extension for providing Ansible auto-completion and integrating quality assurance tools like ansible-lint, ansible syntax check, yamllint, molecule and ansible-test.

aio-libs/aiocassandra 86

Simple threaded cassandra wrapper for asyncio

aio-libs/aiohttp-remotes 70

A set of useful tools for aiohttp.web server

push eventansible-community/nightly-builds

webknjaz

commit sha d8e720402dc943a9e623337ed607c568545195ef

deploy: ansible-community/antsibull@543eee2705c661615bc774b51e54ae82f330ea54

view details

push time in 4 hours

pull request commentjaraco/jaraco.itertools

Add intersphinx mapping to link CPython upstream

Note that RTD didn't use -W.

webknjaz

comment created time in 4 hours

Pull request review commentjaraco/jaraco.itertools

Add intersphinx mapping to link CPython upstream

  # Be strict about any broken references: nitpicky = True++extensions += ['sphinx.ext.intersphinx']+intersphinx_mapping = {+    'python': ('https://docs.python.org/3', None),+}

That sounds reasonable.

webknjaz

comment created time in 5 hours

PullRequestReviewEvent

pull request commentjaraco/jaraco.itertools

Add intersphinx mapping to link CPython upstream

@jaraco I don't see any CI runs of sphinx-build. The easiest thing to do is to go to RTD and select the checkbox for PR builds. I recently noticed that for new sites it is turned on by default.

webknjaz

comment created time in 5 hours

issue commentjaraco/jaraco.itertools

Sphinx error on queue.Queue

I have a static site showing all the linkable objects from select sites. I checked the CPython upstream docs https://webknjaz.github.io/intersphinx-untangled/docs.python.org/ for queue.Queue and found that it exists as a class:

:py:class:`queue.Queue`

This meant that it's linkable via intersphinx. Intersphinx extends the number of objects that Sphinx can reach. But I then noticed that docs.python.org was not added to the mapping explaining why the objects from the site weren't linkable. While adding the mapping, I realized that the extension is not even added. So now I think that #14 should address this issue.

jaraco

comment created time in 5 hours

PR opened jaraco/jaraco.itertools

Add intersphinx mapping to link CPython upstream

Fixes #13

+5 -0

0 comment

1 changed file

pr created time in 5 hours

push eventwebknjaz/jaraco.itertools

Sviatoslav Sydorenko

commit sha efc2be42bdeac37afefbba61c1a5b215e201d6fe

Add intersphinx mapping to link CPython upstream Fixes #13

view details

push time in 5 hours

push eventansible-community/nightly-builds

webknjaz

commit sha e1441fff142102328b6bf985179acb33a9191ca8

deploy: ansible-community/antsibull@543eee2705c661615bc774b51e54ae82f330ea54

view details

push time in 8 hours

pull request commentansible-community/molecule-podman

Prepare an Issue Template

Could you add an issue form instead? Check out ansible/ansible for examples.

sshnaidm

comment created time in 8 hours

pull request commentansible/ansible-navigator

Add ansible-related extras to distribution package

@relrod mostly it's a convenience thing. Helps people not to get themselves into situations like #503 where ansible-core is installed but in the wrong place. Also, think of it as an explicit expression of intent. "I want ansible-navigator with the support of local ansible-core" == pip install ansible-navigator[ansible-core]. When somebody lists direct dependencies of their virtualenv, it's good to make sure that the intent behind adding each of those is clear. It's like a feature flag, in a way. Besides, it'd be aligned with what other projects do.

webknjaz

comment created time in 9 hours

startedDocstringFr/django-tailwind-example

started time in 9 hours

pull request commentpypa/packaging.python.org

Fix broken link

Well, I think you could solve the conflicts / rebase this one and it'd be easier to assess what to do.

meowmeowmeowcat

comment created time in 9 hours

Pull request review commentpypa/packaging.python.org

Fix broken link

     'pip': ('https://pip.pypa.io/en/latest/', None),     'setuptools': ('https://setuptools.readthedocs.io/en/latest/', None),     'twine': ('https://twine.readthedocs.io/en/stable/', None),+    'pypa': ('https://www.pypa.io/en/latest/', None),+    'nox': ('https://nox.thea.codes/en/latest/', None),+    'pynsist': ('https://pynsist.readthedocs.io/en/latest/', None),+    'scipy': ('https://www.scipy.org', None),+    'sphinx': ('https://www.sphinx-doc.org/en/master', None),+    'pybind11': ('https://pybind11.readthedocs.io/en/stable/', None),+    'cffi': ('https://cffi.readthedocs.io/en/latest/', None),+    'python-guide': ('https://docs.python-guide.org', None),+    
meowmeowmeowcat

comment created time in 9 hours

PullRequestReviewEvent

Pull request review commentaio-libs/aiohttp

Adds basic xfail tests for secure proxy support

 import pathlib from unittest import mock +import proxy import pytest from yarl import URL  import aiohttp from aiohttp import web  +@pytest.fixture+def secure_proxy_url(tls_ca_certificate_pem_path, tls_certificate_private_key_path):+    """Return the an instance of a running secure proxy."""+    port = str(proxy.common.utils.get_available_port())+    address = "127.0.0.1"+    input_args = [+        "--num-workers",+        "1",+        "--threadless",+        "--hostname",+        "127.0.0.1",+        "--port",+        port,+        "--cert-file",+        tls_ca_certificate_pem_path,+        "--key-file",+        tls_certificate_private_key_path,+        "--threadless",+    ]+    proxy_url = f"https://{address}:{port}"+    with proxy.Proxy(input_args=input_args):+        yield proxy_url+++async def test_secure_proxy_http_absolute_path(secure_proxy_url, get_request) -> None:+    """Test HTTP urls can be requested through a secure proxy."""+    import pdb++    pdb.set_trace()

Wait until you hear about the $PYTHONBREAKPOINT env var :)

bmbouter

comment created time in 10 hours

PullRequestReviewEvent

Pull request review commentaio-libs/aiohttp

Adds basic xfail tests for secure proxy support

 def read(f):     "pytest-timeout",     "async-generator",     "pytest-xdist",+    "proxy.py",

Let's not do any section removals or packaging refactoring in this PR. It may be reasonable to do this separately, though. Ref: https://github.com/aio-libs/aiohttp/pull/3518#issuecomment-453128368.

bmbouter

comment created time in 10 hours

PullRequestReviewEvent

Pull request review commentaio-libs/aiohttp

Adds basic xfail tests for secure proxy support

 def read(f):     "pytest-timeout",

Looks like you commented here and deleted those comments. Let's not do any section removals or packaging refactoring in this PR.

bmbouter

comment created time in 10 hours

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentaio-libs/aiohttp

Quick fix: Ignore python internal deprecation warning

 addopts = filterwarnings =     error     ignore:module 'ssl' has no attribute 'OP_NO_COMPRESSION'. The Python interpreter is compiled against OpenSSL < 1.0.0. Ref. https.//docs.python.org/3/library/ssl.html#ssl.OP_NO_COMPRESSION:UserWarning+    ignore:The loop argument is deprecated since Python 3.8, and scheduled for removal in Python 3.10.:DeprecationWarning:asyncio.base_events

I think if it improves things at least a little, we should merge it. And yes, we could revert later. Plus I'd combine with #6006 too.

Dreamsorcerer

comment created time in 11 hours

PullRequestReviewEvent

Pull request review commentlektor/lektor

Use setuptools-scm to deduce distribution version and PEP-517-ify the packaging

-[flake8]-ignore=E203,F403,E128,E126,E123,E121,E265,E501,N802,N803,N806,C901,D100,D102,D102,D10,W503,E731,E402,E741+[metadata]+name = Lektor+description = A static content management system.+long_description = file: README.md+long_description_content_type = text/markdown+license = BSD+platforms = any+author = Armin Ronacher+author_email = armin.ronacher@active-4.com+url = http://github.com/lektor/lektor/+classifiers =+    Framework :: Lektor+    Environment :: Web Environment+    Intended Audience :: Developers+    License :: OSI Approved :: BSD License+    Operating System :: OS Independent+    Programming Language :: Python+    Programming Language :: Python :: 3+    Programming Language :: Python :: 3.6+    Programming Language :: Python :: 3.7+    Programming Language :: Python :: 3.8+    Programming Language :: Python :: 3.9+    Topic :: Internet :: WWW/HTTP :: Dynamic Content+    Topic :: Software Development :: Libraries :: Python Modules+requires = setuptools_scm

is this valid?

dairiki

comment created time in 11 hours

push eventansible-community/nightly-builds

webknjaz

commit sha 2691906d9b65d59cee0607577d54965e5a4aae70

deploy: ansible-community/antsibull@543eee2705c661615bc774b51e54ae82f330ea54

view details

push time in 11 hours

PullRequestReviewEvent

Pull request review commentlektor/lektor

Use setuptools-scm to deduce distribution version and PEP-517-ify the packaging

 paths = # NB: (Pinned) pylint is installed as part of the lektor[test] extras commands =     pylint {posargs:lektor setup.py tests}++[flake8]+max-line-length = 100

Is this the current standard? If so, it's not very friendly accessibility-wise. It may cause poor developer experience for people who use large fonts and vertical panels with 2-3 files open side-by-side sometimes.

dairiki

comment created time in 12 hours

Pull request review commentlektor/lektor

Use setuptools-scm to deduce distribution version and PEP-517-ify the packaging

 paths = # NB: (Pinned) pylint is installed as part of the lektor[test] extras commands =     pylint {posargs:lektor setup.py tests}++[flake8]+max-line-length = 100+ignore =
  1. use
extend-ignore =

to append to the default list of ignores instead of replacing it. 2) it's usually much easier to find this config if it's in a dedicated file

dairiki

comment created time in 12 hours