Skip to content
This repository was archived by the owner on Oct 17, 2021. It is now read-only.

Conversation

@JustinVenus
Copy link
Collaborator

Changes

Minimal changes to upgrade to pants 1.26.0. Some of the noisiest deprecation messages have been addressed in this change. Next steps will be to fix the remaining deprecation messages to prepare to catch up to the latest version of pants.

Testing

% ./pants --version 2>/dev/null
1.26.0

% ./pants filter --filter-type=python_tests $(./pants list ::) | xargs ./pants test
...
20:42:01 00:05     [pytest-prep]
20:42:02 00:06     [pytest]
                   src/test/python/apache/aurora/admin                                             .....   SUCCESS
                   src/test/python/apache/aurora/client                                            .....   SUCCESS
                   src/test/python/apache/aurora/client/api                                        .....   SUCCESS
                   src/test/python/apache/aurora/client/cli                                        .....   SUCCESS
                   src/test/python/apache/aurora/client/docker                                     .....   SUCCESS
                   src/test/python/apache/aurora/client/hooks                                      .....   SUCCESS
                   src/test/python/apache/aurora/common                                            .....   SUCCESS
                   src/test/python/apache/aurora/common/health_check                               .....   SUCCESS
                   src/test/python/apache/aurora/config                                            .....   SUCCESS
                   src/test/python/apache/aurora/tools                                             .....   SUCCESS
20:42:02 00:06     [junit]
               Waiting for background workers to finish.
20:42:02 00:06   [complete]
               SUCCESS

@JustinVenus JustinVenus requested a review from ridv June 16, 2020 01:47
Copy link
Contributor

@ridv ridv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, fantastic effort!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants