Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented --force for fetch/pull. Force using +src:dest #1142

Merged
merged 1 commit into from
Apr 28, 2024

Conversation

juanmv94
Copy link
Contributor

No description provided.

Copy link

netlify bot commented Apr 28, 2024

Deploy Preview for xenodochial-hugle-b9ec84 ready!

Name Link
🔨 Latest commit d1a0832
🔍 Latest deploy log https://app.netlify.com/sites/xenodochial-hugle-b9ec84/deploys/662e14bf33f65a0008190618
😎 Deploy Preview https://deploy-preview-1142--xenodochial-hugle-b9ec84.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pcottle
Copy link
Owner

pcottle commented Apr 28, 2024

Awesome! Do you have a good command that would demonstrate the force functionality? was going to add a test...

@pcottle pcottle merged commit 2749d9c into pcottle:main Apr 28, 2024
5 checks passed
pcottle pushed a commit that referenced this pull request Apr 28, 2024
@pcottle
Copy link
Owner

pcottle commented Apr 28, 2024

@juanmv94 I added a basic test case for this in 6cbd02e, here's how I did it:

  • Opened up the sandbox and ran a few commands that exercised the new code
  • Opened up the JS console and ran this command to copy the tree string
  • Added it to the spec file
Screenshot 2024-04-28 at 9 09 44 AM

Can you think of a set of commands that would only work with git pull --force and maybe the +src syntax?

@pcottle
Copy link
Owner

pcottle commented Apr 28, 2024

Oh and that command pops up a dialog where you can copy the string easily:
Screenshot 2024-04-28 at 9 13 13 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants