Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add switch to OKify to stop copying testref files to file.old #49

Open
cdsontag opened this issue Feb 7, 2019 · 3 comments
Open

Add switch to OKify to stop copying testref files to file.old #49

cdsontag opened this issue Feb 7, 2019 · 3 comments

Comments

@cdsontag
Copy link
Collaborator

cdsontag commented Feb 7, 2019

@oiintam and I realized (again) this am that the OKify process needlessly copies old testref files to a spot locally on disk.

Since the "old" testref files are already in the repo by that time, we (ETC) don't need copies made of them.

@cdsontag
Copy link
Collaborator Author

cdsontag commented Feb 7, 2019

CC @cslocum @ariedel

@vglaidler
Copy link
Collaborator

Note that we want to do this as a switch, not by simply deleting code, because we want it to still function correctly for the non-repo case.

@cslocum
Copy link
Contributor

cslocum commented Feb 8, 2019

I brought this up several years ago and Vicki resisted...

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

No branches or pull requests

3 participants