Skip to content

Removed unnecessary files from main - #29

Merged
cangokmen merged 1 commit into
mainfrom
cangokmen/cleanup_main
Aug 24, 2025
Merged

Removed unnecessary files from main#29
cangokmen merged 1 commit into
mainfrom
cangokmen/cleanup_main

Conversation

@cangokmen

Copy link
Copy Markdown
Contributor

Deleted files: test.sh, trees/QuART_stail.h, workloads/10000000_sorted.txt, workloads/createdata_N1000_K10_L10_S1234_a1_b1_P252.txt

test.sh: It was a script that I used to test stail on all the workloads; it wasn't supposed to be pushed
trees/QuART_stail.h: should have been pushed in my next PR
workloads/*: These were Noah's manually created workload files; they weren't supposed to be pushed

@cangokmen cangokmen self-assigned this Aug 21, 2025
@cangokmen cangokmen added the invalid This doesn't seem right label Aug 21, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes unnecessary files that were accidentally pushed to main, cleaning up the repository by deleting development artifacts and premature implementation code.

  • Removes test script and workload files that weren't intended for the main branch
  • Deletes premature QuART_stail header implementation
  • Cleans up accidentally committed development artifacts

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
test.sh Removes testing script for workload execution
trees/QuART_stail.h Removes premature QuART_stail class implementation
workloads/10000000_sorted.txt File deletion (not shown in diff)
workloads/createdata_N1000_K10_L10_S1234_a1_b1_P252.txt File deletion (not shown in diff)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ramananeesh ramananeesh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@cangokmen
cangokmen merged commit 38e3d6d into main Aug 24, 2025
1 check passed
@cangokmen
cangokmen deleted the cangokmen/cleanup_main branch August 24, 2025 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants