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

Glasgow 6 - Haroun- JS Core 1 Week 3 - #230

Open
HarounAlarabi wants to merge 3 commits into
CodeYourFuture:mainfrom
HarounAlarabi:main
Open

Glasgow 6 - Haroun- JS Core 1 Week 3#230
HarounAlarabi wants to merge 3 commits into
CodeYourFuture:mainfrom
HarounAlarabi:main

Conversation

@HarounAlarabi

Copy link
Copy Markdown

No description provided.

@Muath-Alawadhi Muath-Alawadhi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

well done, tidy and easy to read


// TODO
}

@Muath-Alawadhi Muath-Alawadhi Mar 9, 2023

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You did well! You must remove // TO DO next time

}
return headlines;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great using /\d/ .test

Comment thread 2-mandatory/3-stocks.js

}
// TODO

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

same here

Comment thread 2-mandatory/3-stocks.js



// TODO

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WELL DONE!

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.

3 participants