Skip to content
This repository was archived by the owner on Sep 21, 2021. It is now read-only.

10_Risk_Evaluator_Final_PR #182

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

10_Risk_Evaluator_Final_PR #182

wants to merge 22 commits into from

Conversation

e-south
Copy link
Contributor

@e-south e-south commented Apr 29, 2021

Description of Change

Submitting final code and report!

Team Name and Contributors

Eric South, Nick Mosca, Evie Wan

General

  • PR does not include irrelevant or hidden files (i.e. __pycache__, .vscode... etc.)
  • Changes are in the correct folder
  • There are no merge conflicts
  • Commit messages are sensible and descriptive

Code

  • Implementation matches proposed design
  • Documentation on how to run the code is created/updated
  • Code is legible

Report

  • Report meets the deliverable criteria
  • Proposal is up to date with latest decisions

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants