Skip to content

Rename build_tools directory to dev_tools #719

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

Merged
merged 3 commits into from
May 26, 2025
Merged

Conversation

mhucka
Copy link
Collaborator

@mhucka mhucka commented May 24, 2025

The build_tools directory has only one thing in it, and is not even mentioned in any qsim documentation. Other Quantumlib projects use dev_tools for the same thing. I think it would be better for long-term maintenance to rename build_tools to be dev_tools.

The `build_tools` directory has only one thing in it, and is not even
mentioned in any qsim documentation. Other Quantumlib projects use
`dev_tools` for the same thing. I think it would be better for long-term
maintenance to rename `build_tools` to be `dev_tools`.
@mhucka mhucka self-assigned this May 24, 2025
@mhucka mhucka marked this pull request as ready for review May 24, 2025 01:46
@mhucka mhucka requested review from pavoljuhas and dstrain115 May 24, 2025 01:46
@mhucka mhucka added area/devops Involves build systems, Make files, Bazel files, continuous integration, and or other DevOps topics area/health Involves general matters of project configuration, health, maintenance, and similar concerns labels May 24, 2025
@mhucka mhucka merged commit 280aa89 into master May 26, 2025
13 of 18 checks passed
@mhucka mhucka deleted the mh-rename-build-tools branch May 26, 2025 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devops Involves build systems, Make files, Bazel files, continuous integration, and or other DevOps topics area/health Involves general matters of project configuration, health, maintenance, and similar concerns
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants