Skynet/wiki
22
1
Fork 8
Code Issues 1 Pull requests Projects Wiki Activity Actions

docs: Added section describing alternate PR review actions #42

Closed
nanda wants to merge 3 commits from (deleted):main into main
pull from: (deleted):main
merge into: Skynet:main
Conversation 4 Commits 3 Files changed 5 +31 -1
nanda commented 2026-03-23 01:16:41 +00:00
Owner
Copy link
No description provided. 
Reviewed-on: nanda/wiki#1
silver requested changes 2026-03-23 07:39:09 +00:00
Dismissed
silver left a comment
Owner
Copy link

image

Ye might need to do git lfs push origin main to get teh files uploaded.
(LFS is installed right?)

Also would be good to not have teh Merge commit in it

![image](/attachments/0ed5f783-6737-4a61-9a7f-487f4887818f) Ye might need to do ``git lfs push origin main`` to get teh files uploaded. (LFS is installed right?) Also would be good to not have teh Merge commit in it
image.png
216 KiB
Merge branch 'main' into main
All checks were successful
/ check_lfs (pull_request) Successful in 3s
Details
8b3bb76852
silver requested changes 2026-03-24 20:03:13 +00:00
silver left a comment
Owner
Copy link

Could ye please get rid of the two merging commits

image

Could ye please get rid of the two merging commits ![image](/attachments/be0e53ef-c983-46cb-a6ed-0cab18db3f92)
image.png
119 KiB
nanda closed this pull request 2026-03-28 23:13:53 +00:00
silver commented 2026-03-28 23:40:33 +00:00
Owner
Copy link

Due to PBCAK this is now continued in #43

Notable issues was that one of the iamges was not clear in teh slightest.

Due to PBCAK this is now continued in #43 Notable issues was that one of the iamges was not clear in teh slightest.
nanda commented 2026-03-28 23:44:42 +00:00
Author
Owner
Copy link

@silver wrote in #42 (comment):

PBCAK

I'll have you know this was entirely planned and intentional mhm yup.
They call me the 10x engineer because I'm 10xing my PM's heart rate.

@silver wrote in https://forgejo.skynet.ie/Skynet/wiki/pulls/42#issuecomment-3109: > PBCAK I'll have you know this was entirely planned and intentional mhm yup. They call me the 10x engineer because I'm 10xing my PM's heart rate.
silver locked and limited conversation to collaborators 2026-03-28 23:46:19 +00:00
silver unlocked this conversation 2026-03-28 23:47:31 +00:00
silver locked as Too heated and limited conversation to collaborators 2026-03-28 23:49:41 +00:00
silver unlocked this conversation 2026-03-28 23:51:30 +00:00
All checks were successful
/ check_lfs (pull_request) Successful in 3s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
compsoc
silver
Labels
Clear labels   
bug

   
documentation

   
duplicate

   
enhancement

   
good first issue

   
help wanted

   
invalid

   
question

   
wontfix
  
Trainee - Good First Issue

Something that is good for first issue

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Trainee - Good First Issue
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
Skynet/wiki!42
No description provided.