Skip to content
This repository was archived by the owner on Mar 5, 2024. It is now read-only.

Conversation

CPQE
Copy link

@CPQE CPQE commented Nov 15, 2020

someone to walk me through some of it because the C++ style is kind of confusing.

This will be my first PR so please give me feedback.

…someone to walk me through some of it because the C++ style is kind of confusing
Copy link

@SammCheese SammCheese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good, a few misspellings but overall fairly understandable

Note: try to keep opinions out of the comments and instead list them in Issues (MainWindow.cpp - Line 318- 320)

@ClusterConsultant ClusterConsultant added Priority: Medium This issue may be useful, and needs some attention. Type: Documentation Improvements or additions to documentation labels Nov 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Priority: Medium This issue may be useful, and needs some attention. Type: Documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants