Skip to content

chore: remove typos, spelling mistakes & trailing whitespaces#16

Open
jaques-sam-tlv wants to merge 1 commit into
rdjondo:mainfrom
jaques-sam-tlv:fix/solve_typos_spelling_and_trailing_whitespace
Open

chore: remove typos, spelling mistakes & trailing whitespaces#16
jaques-sam-tlv wants to merge 1 commit into
rdjondo:mainfrom
jaques-sam-tlv:fix/solve_typos_spelling_and_trailing_whitespace

Conversation

@jaques-sam-tlv

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread README.md

## Windows
Visit https://visualstudio.microsoft.com/vs/community/ and download Visual Studio Community.
Visit https://visualstudio.microsoft.com/vs/community/ and download Visual Studio Community.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What is the change here?

Comment thread README.md

### In Windows
Visual Studio will let you choose the specific exercise you would like to solve.
Visual Studio will let you choose the specific exercise you would like to solve.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What changed?

// even after you already figured it out.

int type_function(int input) {
x = input;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What changed on this?


// This example showcases the use of a constexpr (computable at compile time)
constexpr int double_var(const int x){
// Solution fix the next line variable : note input cannot be modified

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why removing this?


int double_value_function(int x) {
return double_var(x);
return double_var(x);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What has changed?


// We sometimes encourage you to keep trying things on shape given exercise,
// even after you already figured it out.
// even after you already figured it out.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is the change?


public:
Holder(std::string n = "") {
Holder(std::string n = "") {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is the change?

// We sometimes encourage you to keep trying things on shape given exercise,
// even after you already figured it out.
// even after you already figured it out.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is the change?


// We sometimes encourage you to keep trying things on shape given exercise,
// even after you already figured it out.
// even after you already figured it out.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is the change?

// even after you already figured it out.

// Step 1: Make me compile. Share ownership of data from fill_vec1 to fill_vec2.
// Step 1: Make me compile. Share ownership of data from fill_vec1 to fill_vec2.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is the change?

@rdjondo

rdjondo commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Can you review all your changes and reduce the amount of unnecessary changes?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants