Skip to content

cleanup: Get rid of some old files that have no use#5234

Merged
lgritz merged 1 commit into
AcademySoftwareFoundation:mainfrom
lgritz:lg-lsan
Jun 15, 2026
Merged

cleanup: Get rid of some old files that have no use#5234
lgritz merged 1 commit into
AcademySoftwareFoundation:mainfrom
lgritz:lg-lsan

Conversation

@lgritz

@lgritz lgritz commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

python_tests was from 2010 and either was checked in inadvertently, or has long, long been supplanted by our real testsuite feature.

The various TRAVIS_SKIP_LSAN file used to mark certain tests as skippable when running the leak sanitizer... but we long ago removed the code in runtest.py that recognized its presence and took action. So it has served no purpose for quite some time.

@lgritz lgritz requested a review from grdanny June 13, 2026 23:48

@grdanny grdanny left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

other than the ci fail (which was cause by me - sorry)
this lgtm

python_tests was from 2010 and either was checked in inadvertently, or
has long, long been supplanted by our real testsuite feature.

The various TRAVIS_SKIP_LSAN file used to mark certain tests as
skippable when running the leak sanitizer... but we long ago removed
the code in runtest.py that recognized its presence and took
action. So it has served no purpose for quite some time.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz lgritz merged commit ffc84fa into AcademySoftwareFoundation:main Jun 15, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants