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

Cleanup #902

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Cleanup #902

merged 2 commits into from
Sep 30, 2024

Conversation

LeroyR
Copy link
Member

@LeroyR LeroyR commented Sep 9, 2024

We currently have some unused tasks in the tests as well as the tasks folder.
The repository layout could be confusing for .. anyone.

This moves all unused tasks into tasks/old for lookup. They should just be deleted (we use git after all) but for convenience i just moved the files for now.

Also, the other commit is adding build instructions.

@MatthijsBurgh
Copy link
Collaborator

MatthijsBurgh commented Sep 9, 2024

We have version control, so we don't need an old folder. Just delete them. If you want stuff to be easy to find, I suggest to tag the commit before deleting stuff.

Copy link
Member

Choose a reason for hiding this comment

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

wow that is confusing that this was in both tests and tasks.

@LeroyR LeroyR merged commit d82e3f6 into master Sep 30, 2024
10 checks passed
@LeroyR LeroyR deleted the cleanup_old_tests branch September 30, 2024 07:56
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.

5 participants