JohnCosta27/GitGud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat/eng-123-initializing-project #1

Merged
JohnCosta27 merged 3 commits from feat/eng-123-initializing-project into main 2025-10-12 20:38:49 +01:00
Conversation 3 Commits 3 Files Changed 1 +16 -3
JohnCosta27 commented 2025-10-12 20:14:26 +01:00
Owner
Copy Link
No description provided.
JohnCosta27 added 2 commits 2025-10-12 20:14:26 +01:00
chore: reorganizing initialization a4ead847c3
whoops fcfc3ec8a4
JohnCosta27 force-pushed feat/eng-123-initializing-project from fcfc3ec8a4 to 00a5d2454b 2025-10-12 20:19:00 +01:00 Compare
JohnCosta27 force-pushed feat/eng-123-initializing-project from 00a5d2454b to 47bb47ace4 2025-10-12 20:26:30 +01:00 Compare
JohnCosta27 added 1 commit 2025-10-12 20:27:13 +01:00
feat: adding another user greeting fc38677d84
JohnCosta27 reviewed 2025-10-12 20:28:21 +01:00
index.ts Outdated
@ -6,0 +7,4 @@
function init() {
console.log(`hello ${PERSON}!`);
console.log("another log!");
JohnCosta27 commented 2025-10-12 20:28:21 +01:00
Author
Owner
Copy Link

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
JohnCosta27 reviewed 2025-10-12 20:29:09 +01:00
index.ts Outdated
@ -1,5 +1,16 @@
console.log("hello youtube!");
const PERSON = "youtube";
JohnCosta27 commented 2025-10-12 20:29:09 +01:00
Author
Owner
Copy Link

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 20:31:39 +01:00 Compare
JohnCosta27 force-pushed feat/eng-123-initializing-project from 2ac458dd37 to af1b20c433 2025-10-12 20:33:14 +01:00 Compare
JohnCosta27 added 1 commit 2025-10-12 20:35:19 +01:00
fixup! feat: adding another user greeting 5ad483620d
JohnCosta27 reviewed 2025-10-12 20:36:04 +01:00
JohnCosta27 left a comment
Author
Owner
Copy Link

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 20:37:58 +01:00 Compare
JohnCosta27 merged commit c50390db00 into main 2025-10-12 20:38:49 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
JohnCosta27
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: JohnCosta27/GitGud#1
No description provided.
Delete Branch "feat/eng-123-initializing-project"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?