Skip to content

Conversation

boomanaiden154
Copy link
Contributor

Top level directories should probably be reserved for majorish efforts. I think the ops-container definitions make a lot more sense to have under the premerge folder even if they are not perfectly aligned.

@boomanaiden154 boomanaiden154 requested a review from lnihlen July 10, 2025 21:44
@boomanaiden154
Copy link
Contributor Author

@jriv01

@cmtice
Copy link
Contributor

cmtice commented Aug 21, 2025

I just noticed that this PR is still unreviewed. Is this work still something you want to commit? Should I review it or do you just want to close it?

Top level directories should probably be reserved for majorish efforts. I think
the ops-container definitions make a lot more sense to have under the premerge
folder even if they are not perfectly aligned.
@boomanaiden154
Copy link
Contributor Author

I just noticed that this PR is still unreviewed. Is this work still something you want to commit? Should I review it or do you just want to close it?

I think this is still something we want to do.

@boomanaiden154 boomanaiden154 requested a review from jriv01 August 21, 2025 17:25
@boomanaiden154 boomanaiden154 merged commit c0f449d into llvm:main Aug 21, 2025
6 of 8 checks passed
@boomanaiden154 boomanaiden154 deleted the move-ops-container branch August 21, 2025 17:40
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.

3 participants