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.

Sign up for GitHub

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

Jump to bottom

Enforce clang formatting #207

Open

JRPan opened this issue May 10, 2023 · 1 comment

Assignees

Collaborator

JRPan commented May 10, 2023

No description provided.

JRPan assigned FJShen

Contributor

FJShen commented May 11, 2023 •

edited

Loading

Plan is to mimic the use of git hooks in GEM5. Python formatting should also be considered. The goal is to standardize the format of committed code.

pre-commit hook must be installed by the programmer. This can be done by adding the installation steps in setup_environment.sh

Format all existing cpp and python code
Lazily check and install pre-commit hooks in setup-env
Enforce format check in Jenkins
Describe formatting requirements in release documentation.

FJShen linked a pull request

that will close this issue

Enforce formatting #235

Closed

This was referenced Jun 26, 2023

Reformat code accel-sim/gpgpu-sim_distribution#60

Open

Reformat code #236

Merged

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