Split up the README into separate documentation files so we can publish
the documentation for git-review.
Change-Id: Idcdf43e73c5ca5b8cb239606236e3b2a660c46b7
This synchronizes requirements.txt, test-requirements.txt and
setup.py with the current state of the openstack/requirements
global-requirements.txt list. Also adds flake8 exclusions to tox.ini
for new hacking rules with which git-review does not yet comply.
Change-Id: I837ba57134bea16dd46c3f512bccd51cb39f76c8
Hard set testr concurrency to 2 so that fewer gerrit instances are
started in parallel compared to the default testr concurrency setting.
This will help avoid contention for memory and process resources when
running the tests.
Change-Id: Id3837500ce243af8d8ef762c6dcf50146876f406
* tox.ini: The LANG, LANGUAGE and LC_ALL environment overrides were
introduced originally during the testr migration in an attempt to be
conservative about the possibility that locale settings in the
calling environment could cause consistency problems for test runs.
In actuality, this should be unnecessary and any place where it does
cause issues ought to be considered an actual bug. Also, having
these in the configuration actively causes older pip to have
problems with non-ASCII content in some package metadata files under
Python 3, so drop it now.
Change-Id: I8747a52141321549c56214710b70c5838aacc84e
Closes-Bug: #1277495
Provide intergation testing for the git-review utility. Requires
Gerrit war file (like 2.6.1-gerrit.war) in the .gerrit directory in
the git-review project (like git-review/.gerrit/2.6.1-gerrit.war).
Tests start local Gerrit and create test user and project there to
be used.
Closes-Bug: 1048724
Change-Id: I3242479dcbcf230085178004540992680f3f8e30
For ease of future source code maintenance by OpenStack developers,
git-review is now adopting the same Python Build Reasonableness as
used by current OpenStack projects.
Closes-Bug: 1179007
Change-Id: I20427b51e84b90dcc1d870a547cdcc5240b4ff0e
* tox.ini: Newer hacking fails to recognize the Apache header in the
git-review script, so pin at <0.6 for now to match an upcoming
refactoring change. This will allow us to get some quick cosmetic
fixes in first.
Change-Id: I8e92b1d23d0b8d87e3bafe43d97c987abdf1d9f8
Fixes bug 1199400.
* tox.ini: In the pep8 environment, flake8 and hacking were both
listed as deps. This was fine while both worked with the latest
pyflakes, but then pyflakes 0.7.3 came along and was getting
installed before the more explicit pyflakes 0.7.2 requirement was
parsed for hacking. Since flake8 is in the requirements list for
hacking anyway, this just removes it from the pep8 environment deps
list.
Change-Id: I647235a4f6d109e00f13a156c57ff3a0e9a13a81
The automagic pypi publishing of releases through jenkins.openstack.org
depends on tox having a testenv called venv. Add this new testenv so
that git-review can be released automagically.
Change-Id: I89d78a22c13f94b5af3cda1115c8068424d42b34
1.17 was the last version that met the basic criteria of helping
first time developers sucessfully make their first commit. That's
the most important task of git-review. Revert all changes since
that version. Unfortunately, some of the minor bug fixes since then
will have to be re-written because 1.18, in addition to introducing
basic bugs, also introduced significant code reorganization.
Addresses bug 1038269.
Re-fixed pep8 errors since we've become more strict since 1.17.
Change-Id: Ia3495a6bb883ddf08155c6bfb69a58d19a15a804
Configure a pep8 tox environment so that git-review can make use of the
standard OpenStack pep8 test job.
Change-Id: I4d655a6fc86bbbf6b594b7b35570e75951aef7ab
We'll process requirements.txt to install depends as needed.
Also include a hacking doc explaining what we expect.
Also, including argparse automatically breaks our automagic version
numbering, so import it in main() (slightly evil, but not terrible)
Change-Id: Ib3f65459fe2cd54ce531b7ae22935e24bd8f8920