gnocchixyz / gnocchi

Timeseries database
Apache License 2.0
299 stars 85 forks source link

Fix `passenv` declaration in tox.ini #1280

Closed rafaelweingartner closed 1 year ago

rafaelweingartner commented 1 year ago

While running the tests locally with the latest tox I was getting the following error message:

pep8: failed with pass_env values cannot contain whitespace, use comma to have multiple values in a single line, invalid values found 'LANG GNOCCHI_TEST_* AWS_*'
  pep8: FAIL code 1 (0.00 seconds)
  evaluation failed :( (0.09 seconds)

That error is happening because of the passenv declaration. This patch is proposing a fix for that.

rafaelweingartner commented 1 year ago

This PR is also going to depend on https://github.com/gnocchixyz/gnocchi/pull/1279

tobias-urdin commented 1 year ago

@mergifyio rebase

mergify[bot] commented 1 year ago

⚠️ This pull request got rebased on behalf of a random user of the organization. This behavior will change on the 1st February 2023, Mergify will pick the author of the pull request instead.

To get the future behavior now, you can configure bot_account options (e.g.: bot_account: { author } or update_bot_account: { author }.

Or you can create a dedicated github account for squash and rebase operations, and use it in different bot_account options.

mergify[bot] commented 1 year ago

rebase

✅ Branch has been successfully rebased

tobias-urdin commented 1 year ago

@mergifyio backport stable/4.4

mergify[bot] commented 1 year ago

backport stable/4.4

✅ Backports have been created

* [#1285 Fix `passenv` declaration in tox.ini (backport #1280)](https://github.com/gnocchixyz/gnocchi/pull/1285) has been created for branch `stable/4.4`