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

made a few comments in the gui files, some may not be correct I need … #229

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

made a few comments in the gui files, some may not be correct I need … #229

wants to merge 1 commit into from

Conversation

cyrusParvereshi
Copy link

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.

made a few comments in the gui files, some may not be correct I need …
666a50f 
…someone to walk me through some of it because the C++ style is kind of confusing
SammCheese
SammCheese approved these changes Nov 15, 2020
View reviewed changes
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.
Reviewers

@SammCheese SammCheese SammCheese approved these changes

Assignees

@PazerOP PazerOP

Labels
Priority: Medium This issue may be useful, and needs some attention. Type: Documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cyrusParvereshi @SammCheese @PazerOP @ClusterConsultant