Adds the options implemented in Spyglass for intermediary validation to
the plugin CLI
Related Change: https://review.opendev.org/#/c/678642/
Depends-On: Iaa385d265b027426f8e5f2376462ffb4c0d1d3fa
Change-Id: I15e348916c14927ff9a5e6a27e55a912f8dd3126
This change loosens pins on the XLS plugin dependencies to closer match
Spyglass. Loosening the pins allows Spyglass to have more choices in
package version when resolving dependencies.
Adds mandatory spyglass dependencies to test-requirementst since
Spyglass' requirements.txt has been removed. This will no longer be
necessary once PBR gets support for Pipfile.
Depends-On: I347ef6db18a44e1b88c811aa071ba8c60bccbd53
Change-Id: I8940b5afe1fce8d572ae90d66f6ae3cbefce9a70
Implements documentation updates from [0] to standardize formatting for
all Airship projects.
[0] https://review.opendev.org/#/c/671291/
Change-Id: I723af0d91e24a306f05c06a3ddf456757489f7f0
Moves the dependency vulnerability checks from pep8 tox check to a new
Zuul job. This allows setting the job to not be voting as the database
used for safety has been flaky recently.
Changes basepython to python3 for safety and bandit jobs.
Related change: https://review.opendev.org/#/c/671847
Change-Id: I3bbc15aec795ff901e442e17d2ab974fd5da90ef
Updates spyglass-plugin-xls to use the updated plugin base in Spyglass.
Related change: https://review.opendev.org/#/c/670898/
Depends-On: Ia3bc892994ff96cce5c1672fadf35ef2d1c4164b
Change-Id: If086d066610dfe2fd0384c7a3453940364ee1608
This change implements the removal of the configuration methods in
Spyglass. Class fields will now be set by the init method instead.
Depends-On: Ib26636f1eb4146902ee801af5bcce53d137be2ad
Change-Id: Ia7e53d900de14d1089b0323ddb08717b07b12de9
The previous excel spec seemed to lack a consistent structure. This
change redesigns the spec to more simply and generically find where data
is located in the spreadsheet.
Change-Id: I98c1553531897e8c623caa8a1b9334ea915a3f49
Sphinx 2.1.0 has a bug [0] that causes whitespace to be excluded in
outputted docs. This change updates Sphinx to peg any version >2.1.0.
Safety dependency vulnerability checks now will also cover the doc
requirements.txt.
[0] https://github.com/sphinx-doc/sphinx/issues/6440
Change-Id: I52b6eac208fd1c99194e2d30b6cf00431550a31d
This change moves spyglass-plugin-xls's necessary exception definitions
locally into the spyglass-plugin-xls project and changes their base
to the SpyglassBaseException class. This change must be merged before
Spyglass's exceptions may be consolidated.
Related Change: https://review.opendev.org/#/c/667240/
Change-Id: I81c5ca2d9083aece3641bc8b5405dfd44baec810
Adds tests for the Excel plugin's parser and extractor.
Enables pep8 and fmt checks on the tests directory.
Increases plugin test coverage to 94%, sets new minimum to 92%.
Fixes DNS and NTP server extraction with regex.
Updates file licenses.
Change-Id: I35ee97574e6d63b7a82cfa94caf79db5db9755e7
This change removes the intermediate step of converting the
SiteDocumentData object returned by the parser to a dictionary. This
change depends on [0], a related change that updates the Jinja2
templates and site processor to use data objects instead of a
dictionary.
[0] https://review.opendev.org/#/c/661092/
Depends-On: I66ebfeaf5d6ca76b6dee5a2285a74bad8b06b720
Change-Id: I991c9f6590752b36050dc5239d9ea3f3dfe8c5bc
This change implements data objects from [0] into the spyglass XLS
plugin. With this change, the intermediary command will fully support
the data objects and documents will partially support them by converting
the objects to dictionaries for consumption in Jinja2.
[0] https://review.opendev.org/#/c/658917/
Depends-On: I101ad1ccbd95822965b8da8b6a644522eb2908e7
Change-Id: Ib95ef56cd8c5f4cddb9ff5c8c862ba7e323c8d49
A recent change updated the settings of yapf in Spyglass. This change
matches this recent change and fixes the directory used in tox. It also
fixes the directories used in tox.ini and removes unnecessary flake8
ignores.
Change-Id: I7036f885a3c45880b7b9be604bc6c1c533622134
Adding PBR into spyglass-plugin-xls has caused a circular dependency.
Because Spyglass installs spyglass-plugin-xls it also now tries to
resolve spyglass-plugin-xls's requirements.txt which in turn requires
Spyglass to be installed. Since Spyglass is what's currently trying to
install, pip fails to resolve the plugin's dependencies. This change
moves the Spyglass requirement into test-requirements.txt. Spyglass will
always install this plugin so spyglass-plugin-xls should only need to
install Spyglass to run its own independent tests.
Change-Id: Ia5765d6b894cc1073c8be803d43f0b90cc448383
This change adds the "force" option to the XLS plugin, allowing users
to forgo manifest missing data validation.
Depends-On: Iff000eb173995156fbc6b44e621c59ba4dffae35
Change-Id: Icc9bf1bef0a6126e0d3915d3c809888c62086ee3
Spyglass uses PBR for its packaging so it makes sense that its plugins
would do the same. This change makes the necessary updates to have
spyglass-plugin-xls use python build reasonableness.
Implements PBR in setup.py and setup.cfg
Moves YAPF configuration into setup.cfg
Change-Id: Ie0dab14bc266a0000566995c8fb2f9640fce7868
This change pins the test-requirements.txt dependencies and upgrades
two packages. PyYAML is upgraded to 5.1 to match Spyglass and openpyxl,
used to read Excel files, is upgraded to 2.6.2. Neither upgrade affects
the behavior of spyglass.
Change-Id: I0383ed56e62f64e2316925d1ba9d241b165ebdde
The examples folder should be kept inside of the package folder as it
won't be included if kept outside in the later versions of setuptools.
Change-Id: I14fb97e2f59571f0b20ae2c6aedfa62244bd6eb5
This change adds dependency vulnerability checks into tox. These checks
are performed by the Safety package which checks requirements files
against a vulnerability database for any issues. If any vulnerabilities
are found, tox will fail its pep8 env.
Change-Id: I6e356d1f55ced7df9490d34e06451acd9cbb6d70
Updates the XLS plugin docs using changes that were to be made by [0].
Changes old references to "tugboat" to "Spyglass XLS plugin".
[0]: https://review.opendev.org/#/c/650137/
Change-Id: I0df6a7f190548d0e0fa9a06d3b644dbc6cfb5280
There were a few missing changes from the repo that this project was
imported from. This change is meant to get the XLS plugin up to date
with any changes that were merged into Spyglass before the separation.
Relates to https://review.opendev.org/#/c/653555/
Change-Id: Ib2f75878b1a29e835cb8e2323aebe9d431c479e7
The source directory for this package was originally named
"spyglass-plugin-xls", which caused it to not be able to be imported by
other projects. This change renames the directory to
"spyglass_plugin_xls" so it can be imported and utilized by Spyglass.
Change-Id: Ida8e7b8dc1d55bdf8cb38066ed72c40c0dcf474d
Add the usual .gitreview file.
Add an initial zuul file so that jobs can merge, this contains only
noop-jobs, so no tests are run. A followup can add proper tests.
Change-Id: Idf5aa1c82419ddab718ca5267d5d250a2dfb64d9
