feat/eng-123-initializing-project #1

Merged
JohnCosta27 merged 3 commits from feat/eng-123-initializing-project into main 2025-10-12 19:38:49 +00:00
Owner
No description provided.
JohnCosta27 force-pushed feat/eng-123-initializing-project from fcfc3ec8a4 to 00a5d2454b 2025-10-12 19:19:00 +00:00 Compare
JohnCosta27 force-pushed feat/eng-123-initializing-project from 00a5d2454b to 47bb47ace4 2025-10-12 19:26:30 +00:00 Compare
index.ts Outdated
@ -6,0 +7,4 @@
function init() {
console.log(`hello ${PERSON}!`);
console.log("another log!");
Author
Owner

This should be in a different commit, don't add exclamation mark here

This should be in a different commit, don't add exclamation mark here
index.ts Outdated
@ -1,5 +1,16 @@
console.log("hello youtube!");
const PERSON = "youtube";
Author
Owner

I would have the commit that extracts this into a function, after the feature commit.

I would have the commit that extracts this into a function, after the feature commit.
JohnCosta27 force-pushed feat/eng-123-initializing-project from fc38677d84 to 2ac458dd37 2025-10-12 19:31:39 +00:00 Compare
JohnCosta27 force-pushed feat/eng-123-initializing-project from 2ac458dd37 to af1b20c433 2025-10-12 19:33:14 +00:00 Compare
JohnCosta27 left a comment

Great! All looking good!

Just apply the fixups!

Great! All looking good! Just apply the fixups!
JohnCosta27 force-pushed feat/eng-123-initializing-project from 5ad483620d to c50390db00 2025-10-12 19:37:58 +00:00 Compare
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
JohnCosta27/GitGud!1
No description provided.