dsi-cocoa 0.5.7

Clinic Opinionated Codebase Oversight and Analysis

Clinic Opinionated Codebase Oversight and Analysis

This repository was developed in order to generate automated reports on how well codebases adhere to the coding standards of the University of Chicago's DSI Clinic course.

The goals of this codebase is to provide a quick and easy way to review code and to alert contributors where their code may be failing.

Installation

pip install dsi-cocoa

To install the package locally, run the following command from the root of the repository:

python3 -m pip install .

cocoa

This package contains a module evaluate_repo which runs code testing libraries against a repository. There are multiple ways that we want to be able to run this repo:

1. We want to be able to clone cocoa and then use it as part of their process.
2. We want administrators to be able to give it a list of repos to generate reports on all repos.
3. (Eventually) We want it to be able to run as a github action on the repo itself.
4. (Eventually) We want to be able to have adminstrative repo that can run it on other repos.

How to run

Via command line:

cocoa /path/to/repo

As a python script:

python3 src/cocoa/evaluate_repo.py /path/to/repo

As a Python module:

from cocoa.evaluate_repo import evaluate_repo

evaluate_repo('/path/to/repo', False)

A few important notes:

1. Make sure to git pull before running this code.
2. This will get branch information for all branches.
3. This will only run the analysis (pyflakes on python files) for the code in the current branch. So if you run this while your current branch is main it will run on main.

Options

Results are truncated by default. To print all results, use the verbose option:

cocoa /path/to/repo --verbose

Cocoa evaluates the main branch by default. To evaluate a different branch, use the branch argument:

cocoa /path/to/repo --branch branch-name

If files modified/created only after a certain date are to be evaluated then use the date option:

cocoa /path/to/repo --date YYYY-MM-DD

Note: Please add date string in YYYY-MM-DD format.

All options can be combined like so:

cocoa /path/to/repo --lint --verbose --date YYYY-MM-DD

Checks

The code run multiple checks on each repo. For each check run there are three possibilities:

1. WARNING: Most likely this needs to be fixed.
2. INFO: Log information for additional context.
3. ERROR: A critical issue that needs to be addressed.

For each of the checks below we have denoted what the check generates.

• Branch Hygiene:
  • [WARNING] Branch names
  • [INFO] Commit information for live branches.
• File Hygiene:
  • [ERROR] Unnecessary and cache file (such as .DS_Store or pycache files)
• Notebook Files (*.ipynb):
  • [ERROR] Cells per notebook < 10, lines per cell < 15 and 0 functions defined
  • [ERROR] Linting: PyLint, Black, Flake and iSort
• Python Files
  • [ERROR] All Code in Functions
  • [ERROR] All functions have docstrings
  • [ERROR] Code uses off-limit libraries (subprocess)
  • [ERROR] Linting: PyLint, Black, Flake and iSort

Github actions

There is a Github action located here that runs cocoa on pushes to the main branch. The action has an associated badge that can be displayed at the top of your repo to show passing or failing status. The badge code can be copied from the raw text of this readme.

To override the --date or --branch options run in the action, create an environment named "cocoa_standards" in your repository, then create environment variables called BRANCH_NAME and REVIEW_AFTER_DATE.