woem.men/forkey
4
2
Fork 3
Code Issues 34 Pull requests 16 Projects Releases Packages Wiki Activity Actions

avoid cloning entire repo for actions/checkout #47

Merged
ashten merged 1 commit from sugar/forkey:avoid-cloning-entire-repo-in-actions-checkout into main 2025-01-19 20:59:35 +00:00
Conversation 1 Commits 1 Files changed 8 -11
sugar commented 2025-01-19 18:44:20 +00:00
Owner
Copy link

fetch-depth: 0 is saying to fetch all commits, which we don't really need test for testing a single commit

What

Why

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests
`fetch-depth: 0` is saying to fetch all commits, which we don't really need test for testing a single commit <!-- ℹ お読みください / README PRありがとうございます！ PRを作成する前に、コントリビューションガイドをご確認ください: Thank you for your PR! Before creating a PR, please check the contribution guide: https://github.com/misskey-dev/misskey/blob/develop/CONTRIBUTING.md --> ## What <!-- このPRで何をしたのか？ どう変わるのか？ --> <!-- What did you do with this PR? How will it change things? --> ## Why <!-- なぜそうするのか？ どういう意図なのか？ 何が困っているのか？ --> <!-- Why do you do it? What are your intentions? What is the problem? --> ## Additional info (optional) <!-- テスト観点など --> <!-- Test perspective, etc --> ## Checklist - [ ] Read the [contribution guide](https://github.com/misskey-dev/misskey/blob/develop/CONTRIBUTING.md) - [ ] Test working in a local environment - [ ] (If needed) Add story of storybook - [ ] (If needed) Update CHANGELOG.md - [ ] (If possible) Add tests
sugar added 1 commit 2025-01-19 18:44:21 +00:00
avoid cloning entire repo for actions/checkout
All checks were successful
Test (production install and build) / production (22.x) (pull_request) Successful in 2m21s
Details
41a9904932 
`fetch-depth: 0` is saying to fetch all commits, which we don't really
need test for testing a single commit
leah commented 2025-01-19 18:48:42 +00:00
Owner
Copy link

Looks good but id like @ashten to take a look as she is the one who knows more about this stuff.

Looks good but id like @ashten to take a look as she is the one who knows more about this stuff.
ashten merged commit f4bda8582f into main 2025-01-19 20:59:35 +00:00
ashten deleted branch avoid-cloning-entire-repo-in-actions-checkout 2025-01-19 20:59:36 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
customization

Change something to be more suitable for forkey

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
feature parity

Another fork has this feature, and it should be pulled into this fork.

  
help wanted

Need some help

  
high priority

   
invalid

Something is wrong

  
low priority

   
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
customization
duplicate
enhancement
feature parity
help wanted
high priority
invalid
low priority
question
wontfix
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: woem.men/forkey#47
No description provided.
Delete branch "sugar/forkey:avoid-cloning-entire-repo-in-actions-checkout"

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?