Skip to content

Commit

Permalink
Remove the scrooge contrib plugin. (pantsbuild#10127)
Browse files Browse the repository at this point in the history
1.30.x is expected to be the last release in the Pants 1.x series. Pants 2.x will no longer support the v1 engine, meaning these tasks will no longer work.

For anyone who still needs this functionality, we will give instructions and a script for users to be able to keep using Pants 1.x and Pants 2.x side-by-side. Given enough user interest, we can also port this functionality to the v2 engine or help users to write a plugin.

[ci skip-rust-tests]
[ci skip-jvm-tests]
  • Loading branch information
stuhood authored Jun 22, 2020
1 parent 3dbb61c commit de6608c
Show file tree
Hide file tree
Showing 75 changed files with 23 additions and 2,361 deletions.
18 changes: 0 additions & 18 deletions 3rdparty/jvm/com/twitter/BUILD

This file was deleted.

5 changes: 0 additions & 5 deletions BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,6 @@ files(
sources = ["BUILD_ROOT"],
)

files(
name = 'build_tools',
sources = ['BUILD.tools'],
)

files(
name = 'gitignore',
sources = ['.gitignore'],
Expand Down
36 changes: 0 additions & 36 deletions BUILD.tools

This file was deleted.

4 changes: 0 additions & 4 deletions build-support/bin/packages.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,10 +91,6 @@ def core_packages() -> Set[Package]:

def contrib_packages() -> Set[Package]:
return {
Package(
"pantsbuild.pants.contrib.scrooge",
"contrib/scrooge/src/python/pants/contrib/scrooge:plugin",
),
Package(
"pantsbuild.pants.contrib.mypy", "contrib/mypy/src/python/pants/contrib/mypy:plugin",
),
Expand Down
1 change: 0 additions & 1 deletion contrib/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ target(
name = 'plugins',
dependencies = [
'contrib/mypy/src/python/pants/contrib/mypy:plugin',
'contrib/scrooge/src/python/pants/contrib/scrooge:plugin',
]
)

Expand Down
3 changes: 1 addition & 2 deletions contrib/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@ plugin APIs are used.

Most new plugins should get their own top-level `contrib/` subdirectory although it may make sense
to house the source code for related plugins under one top-level `contrib/` subdirectory. The
`contrib/scrooge` directory is an example of this and houses two plugin tasks that both use the same
underlying [Scrooge](https://github.com/twitter/scrooge) tool.
`contrib/mypy` directory is an example of this.

Contrib plugins should generally follow 3 basic setup steps:

Expand Down
10 changes: 0 additions & 10 deletions contrib/release_packages.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,6 @@
# `PKG_$NAME` definition.
#

function pkg_scrooge_install_test() {
local version=$1
execute_packaged_pants_with_internal_backends \
--plugins="['pantsbuild.pants.contrib.scrooge==${version}']" \
--explain gen | grep "scrooge" &> /dev/null && \
execute_packaged_pants_with_internal_backends \
--plugins="['pantsbuild.pants.contrib.scrooge==${version}']" \
--explain lint | grep "thrift" &> /dev/null
}

function pkg_mypy_install_test() {
local version=$1
execute_packaged_pants_with_internal_backends \
Expand Down
12 changes: 0 additions & 12 deletions contrib/scrooge/BUILD

This file was deleted.

6 changes: 0 additions & 6 deletions contrib/scrooge/README.md

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

1 change: 0 additions & 1 deletion contrib/scrooge/src/python/pants/__init__.py

This file was deleted.

1 change: 0 additions & 1 deletion contrib/scrooge/src/python/pants/contrib/__init__.py

This file was deleted.

19 changes: 0 additions & 19 deletions contrib/scrooge/src/python/pants/contrib/scrooge/BUILD

This file was deleted.

Empty file.
17 changes: 0 additions & 17 deletions contrib/scrooge/src/python/pants/contrib/scrooge/register.py

This file was deleted.

10 changes: 0 additions & 10 deletions contrib/scrooge/src/python/pants/contrib/scrooge/subsystems/BUILD

This file was deleted.

Empty file.

This file was deleted.

66 changes: 0 additions & 66 deletions contrib/scrooge/src/python/pants/contrib/scrooge/tasks/BUILD

This file was deleted.

Empty file.
Loading

0 comments on commit de6608c

Please sign in to comment.