summaryrefslogtreecommitdiffstats
path: root/hacking (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Use open with context manager (#83337)Abhijeet Kasurde2024-10-181-7/+6
| | | Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update triple single quotes to triple double quotes (#84099)Matt Clay2024-10-091-14/+14
| | | | | | | | | * Update triple single quotes to triple double quotes This change was fully automated. The updated Python files have been verified to tokenize the same as the originals, except for the expected change in quoting of strings, which were verified through literal_eval. * Manual conversion of docstring quotes
* Stop suggesting `easy_install` in hacking (#83909)Sviatoslav Sydorenko (Святослав Сидоренко)2024-09-061-1/+1
| | | | | It's been discouraged for the past decade. And CPython actually ships with pip nowadays, that is bundled within the built-in `ensurepip` stdlib module.
* Remove references to IRC or Google Groups (#83875)John Barker2024-08-294-14/+10
|
* ansible-test - Remove generation of egg-info (#83786)Matt Clay2024-08-142-32/+0
| | | Also remove egg-info generation from hacking/env-setup scripts.
* ansible-test - Update venv management and sanity requirements (#83729)Matt Clay2024-08-061-1/+0
| | | | | | | | | | * ansible-test - Update venv management * Upgrade from pip 24.0 to 24.2 * Omit `wheel` and `setuptools` from ansible-test managed virtual environments * Drop pre-release hacks * Update mypy requirements * Freeze sanity test requirements * Update sanity test configuration * Update sanity ignores
* Remove Python 3.10 support for the controller (#83221)Martin Krizek2024-06-171-1/+1
| | | Fixes #83094
* pymarkdown: update docs (#83150)Abhijeet Kasurde2024-05-0612-33/+35
| | | Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Remove the yum module, redirect it to dnf (#81895)Martin Krizek2024-01-231-1/+1
| | | Fixes #81728
* Cleanup the hacking directory a bit (#81960)Matt Clay2023-10-122-8/+5
| | | | | | | | | * Clean up hacking/ansible-profile script Also rename the script so it has a `.py` extension. * Remove test-module symlink Use the test-module.py script instead.
* Require `from __future__ import annotations` (#81902)Matt Clay2023-10-059-19/+9
|
* env-setup.fish: Correct syntax errors (#81677)Sam Doran2023-09-121-7/+7
| | | | - Single quote regular expression values to avoid shell expansion - Properly test if the $QUIET variable is set and non-zero
* Update update-sanity-requirements.py script (#81424)Matt Clay2023-08-031-7/+6
| | | Frozen requirements can now preserve any explicitly installed package that would normally be omitted, not just setuptools.
* ansible-test - Pre-build PyYAML wheels (#81300)Matt Clay2023-07-191-2/+63
| | | | This works around Cython failures when attempting to install PyYAML >= 5.4 <= 6.0.
* Update env-setup.fish (#81208)Anoint2023-07-121-57/+48
| | | | | | | | | | | | | ## Description This commit includes improvements and optimizations to the script: - Clarified the purpose of the quiet flag with a comment. - Adjusted conditions for appending to PYTHONPATH and MANPATH variables. - Enhanced Python executable check for reliability and cross-platform support. - Improved comments and function clarity. - Ensured consistency with the original code. These changes enhance readability, efficiency, and maintainability of the script.
* Add pymarkdown sanity test for core (#81220)Matt Clay2023-07-1114-59/+74
| | | | | | | * Clean up markdown * Add pymarkdown sanity test for core * Update unit test SHA
* Remove `docs` and `examples` directories (#81011)Matt Clay2023-07-1115-1577/+0
| | | | | | | | | | | | | | | | | | | | | * Remove docs dir * Updates to reflect docs removal * Fix integration test * Remove examples dir * Updates to reflect examples removal * Remove build_library and build-ansible.py * Remove refs to build_library and build-ansible.py * Remove obsolete template * Remove obsolete template reference * Remove the now obsolete rstcheck sanity test
* Remove Python 3.9 support for the controller (#80973)Sloane Hertel2023-07-101-1/+1
| | | | | | | * Remove obsolete Python <=3.9 controller code * Remove Python 3.9 test controller bootstrapping * Update test requirements Co-authored-by: Matt Clay <matt@mystile.com>
* Remove long obsolete hacking/get_library.py (#81122)Matt Clay2023-06-241-34/+0
|
* Remove hacking/fix_test_syntax.py (#81121)Matt Clay2023-06-242-121/+0
| | | This script is no longer needed, since using tests as filters was removed in 2.9.
* Revert "Add a custom policy for hackers using ansible-test (#68535)" (#81120)Matt Clay2023-06-243-48/+0
| | | This reverts commit bd9e31fcb01100a510653f6f49715833bd637d31.
* Remove docs dir dependency from man page build (#81003)Matt Clay2023-06-082-1/+128
|
* No core irc meeting (#80811)Matt Martz2023-05-161-4/+0
|
* Use ansible.module_utils.common.text.converters (#80704)Matt Clay2023-05-037-7/+7
| | | Replace use of old `ansible.module_utils._text` and add a unit test to maintain backwards compatibility.
* Set the minimum setuptools to 45.2.0 (#80649)Matt Clay2023-04-261-1/+13
| | | Also update the package-data sanity test to use the minimum setuptools version.
* Remove obsolete pkg_resources usage (#80489)Matt Clay2023-04-121-11/+0
|
* Fix misrendered sections in manpage generationSviatoslav Sydorenko2023-04-081-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change fixes bugs in the manpage generator that existed since it was first added. It exposes CLI `ARGUMENTS` value to manpage templates. Before this change, the code contained a typo, causing the `for`-loop iterate over individual characters of the `'ARGUMENTS'` string rather than iterating over a tuple. A missing comma was at fault. The updated code gets rid of the `for`-loop and conditionals since it seems to have been a premature complexity increase and no other things than `'ARGUMENTS'` were ever added into the broken iterable. The functional change is that `arguments` is now always present in the Jinja2 context, unlike being missing sometimes because of the previous design (not that it was ever present, because of the bug! sigh...) The Jinja2 templates perform an `{% if arguments %}` check, letting the template engine silently ignore the missing variable. The clause was always falsy, meaning that the arguments section was not included in the manpages for at least the last 6 years. With this fix, it will be. This patch also deduplicates calling `opt_doc_list` @ generate_man. It was called late in the execution, more times than necessary. This patch makes sure it happens once by putting it at the top of the scope. It fixes rendering library and inventory in manpages. The corresponding Jinja2 templates have blocks wrapped with conditionals like `{% if inventory %}` and `{% if library %}` but said variables were never injected into the context, nor were they even deduced on the Python side of the generator. This means that the conditional clauses were always falsy, never showing the portions of the manpages. The Python script has hints for how the `inventory` variable was to be calculated, which is confirmed through the Git paleontology efforts. The block of code that references to the `inventory` bit was incorrectly checking a variable with a list of nested objects for the presence of a string which was never going to work. This patch fixes this check by verifying the CLI flag against the correct variable containing a list of options and exposes it to the Jinja2 templates. It also exposes the `library` variable in a similar way. The block displaying other binaries in Sphinx CLI docs has been synchronized with the manpage template. Previously, the current binary was displayed also. This patch gets rid of the unwanted trailing comma there too. Finally, the CLI executables list in the manpage template now reuses the same variable as the RST template that doesn't need any post-processing in Jinja2. Before, it was already used in the RST template so this patch aligns both templates to use the same logic as they got out-of-sync over time. PR #80450.
* Extend bulk issue creator (#80423)Matt Clay2023-04-061-9/+164
| | | It can now be used to create feature requests, not just deprecation bug reports.
* Remove old release announcement scripts (#80371)Matt Clay2023-04-042-371/+0
| | | These have been replaced by `packaging/release.py`
* Add work-around for old AZP jobs (#80339)Matt Clay2023-03-281-0/+8
|
* Remove straight.plugin dependency (#80084)Matt Clay2023-02-241-3/+30
|
* Clean up unused imports in `hacking` directory (#79894)Matt Clay2023-02-033-8/+1
|
* Flush output in sanity requirements update script. (#79774)Matt Clay2023-01-201-1/+1
| | | This keeps the subprocess output under the correct heading.
* Add support for importlib.resources (#78915)Matt Martz2023-01-101-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add support for importlib.resources * Remove the importlib.resources imports * return the correct data * Some code comments, and re-order for consistency * Disallow traversing packages below an individual collection * Add a traversable class for namespaces * Re-use variable * Utilize itertools.chain.from_iterable Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua> * Simplify logic to check for packages from ansible loaders Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua> * Just a generator expression, instead of a generator * docstrings * Add comment about find_spec for our namespaces * Add some initial unit tests for importlib.resources * normalize * Utilize importlib.resources for listing collections * collections_path is already in config, just use config * install uses a different default for collections_path * Remove unused import * Remove duplicate __truediv__ * Bring back TraversableResources * Apply some small suggestions from code review Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua> Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com> * Remove cross contamination between plugin loader code and CLI code * Remove unused import Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua> Co-authored-by: Matt Davis <6775756+nitzmahone@users.noreply.github.com>
* stable docs build: do not use YAML parsing/serialization for .deps files ↵Felix Fontein2022-10-261-6/+22
| | | | | | | (#79233) * Do not use YAML parsing/serialization for .deps files. * Prevent crash.
* Fix distro fact handling for Flatcar (#77635)Johanan Liebermann2022-09-071-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | * Fix distro fact handling for Flatcar The existence of the file /etc/flatcar/update.conf depends on bootstrap configuration typically provided by the user. For that reason this file is unsuitable for determining distro facts for Flatcar Container Linux. The distribution_release fact is meaningless in the case of Flatcar since Flatcar doesn't have named releases. The distribution_version fact, however, IS meaningful and should contain a number such as "3139.2.0". - Use /etc/os-release instead of /etc/flatcar/update.conf. - Drop the distribution_release fact. - Set the distribution_version fact. - Update distro test fixture for Flatcar - Generate the fixture using gen_distribution_version_testcase.py. - Override result.distribution and result.os_family manually as the generator script gives wrong values. - Use a recent Flatcar version. Signed-off-by: Johanan Liebermann <jliebermann@microsoft.com>
* Add missing space after keywords. (#78480)Matt Clay2022-08-081-1/+1
| | | Also remove unnecessary parenthesis.
* `FieldAttribute`s as descriptors (#73908)Martin Krizek2022-06-291-2/+2
|
* Update README.md (#78139)Diana S. Cardona2022-06-281-2/+1
| | | Update Python version requirement to >= 3.8.
* Allow result sha to be overriden with local sha (#77832)Matt Martz2022-05-181-5/+11
|
* Stop making core docs build talk of (ansible-)'base' when meaning ↵Felix Fontein2022-05-111-16/+16
| | | | (ansible-)'core'. (#77728)
* shift to ansible_core_versionsamccann2022-05-031-2/+2
|
* Fix use of deprecated antsibull-docs option.Matt Clay2022-04-251-2/+2
|
* Replace antsibull with antsibull-docs (#77504)Felix Fontein2022-04-222-3/+3
|
* Make "~/.ansible" path configurable (#76114)htol2022-04-111-8/+8
| | | | | | | | | | * replace hardcoded '~/.ansible' to C.ANSIBLE_HOME * rename previously existing env ANSIBLE_HOME in env-setup script * modify cache dir monkeypatching in galaxy api unit tests * update "version_added" to 2.14 for ANSIBLE_HOME * fix description of collections with proper use of ANSIBLE_HOME Co-authored-by: htol <github@h-tol.net> Co-authored-by: stefanwascoding <stefan@syntaxhelden.de>
* Add script to handle more deprecations. (#77400)Matt Clay2022-03-301-0/+285
| | | | | | | | | | * Add script to handle more deprecations. This script currently supports deprecations from the following sanity tests: * deprecated-config * update-bundled * Ignore script in package-data test.
* Update core porting guide generation.Matt Clay2022-03-281-6/+6
|
* Porting guide fixes. (#77376)Matt Clay2022-03-281-2/+2
| | | | * Update porting guide generation. * Fix 2.13 porting guide link.
* Improve documentation to fix outdated file paths (#77341)Cindy Park2022-03-242-5/+5
| | | Co-authored-by: njthanhtrang <njthanhtrang@users.noreply.github.com>
* Honor `--quiet` parameter properly in fish shell (#77180)Philip Douglass2022-03-041-8/+3
| | | | * The `^` stderr redirection is deprecated * Setting `-q` for `setup.py` in `gen_egg_info` is redundant