This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Remove scripts/config.ini #189

Open

mikelik opened this issue May 23, 2023 · 1 comment

Labels

👍 lgtm tech debt

Contributor

mikelik commented May 23, 2023

1.After #184 was merged I think we don't longer need scripts/config.ini so it can be removed.

Maybe tests/config.ini can also be removed?

The text was updated successfully, but these errors were encountered:

mikelik added the tech debt label

enf-ci-bot added the triage label

enf-ci-bot added this to Team Backlog

github-project-automation bot moved this to Todo in Team Backlog

Contributor

ScottBailey commented May 23, 2023

tests/config.ini is used to test that an external config can be used. So we either want to keep it or we want to write a config in the tests.

stephenpdeos added 👍 lgtm and removed triage labels

bhazzard moved this from Todo to Icebox in Team Backlog

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