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

Added PATCH method #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fpvmorais
Copy link

@fpvmorais fpvmorais commented Mar 27, 2024

Hello,

This is just a tiny contribution, I'm currently running Emacs on a Windows SO and needed to create pull requests on DevAzure.
One of the endpoints demands the PATCH method.

Being using this for a few weeks and works properly.

One issue I've found is that it does not work with cURL from Microsoft located on C:\Windows\System32\curl.exe. (Version 8.4.0). Not sure exactly why.
After installing the Curl version (8.6.0) via Scoop and setting this variable on Emacs:
(setq plz-curl-program "C:\\Tools\\scoop\\apps\\curl\\current\\bin\\curl.exe")
Everything started working correctly. Not sure if it's worth it to add to the documentation, but it might help someone in the same situation as me.

Thank you so much for your amazing packages and work.
All the best from Portugal

Pedro

@alphapapa
Copy link
Owner

Hi Pedro,

Hello,

This is just a tiny contribution, I'm currently running Emacs on a Windows SO and needed to create pull requests on DevAzure. One of the endpoints demands the PATCH method.

Being using this for a few weeks and works properly.

Thanks, this looks good. And thanks for including a test. I'll plan to add this in the next version.

One issue I've found is that it does not work with cURL from Microsoft located on C:\Windows\System32\curl.exe. (Version 8.4.0). Not sure exactly why. After installing the Curl version (8.6.0) via Scoop and setting this variable on Emacs: (setq plz-curl-program "C:\\Tools\\scoop\\apps\\curl\\current\\bin\\curl.exe") Everything started working correctly. Not sure if it's worth it to add to the documentation, but it might help someone in the same situation as me.

Yes, Windows's "ersatz curl" presents some problems with argument handling compared to the real thing. We should mention that in the docs somewhere. Feel free to open an issue about that if you like.

By the way, this probably fits within the "tiny change" guidelines, but it would be good to complete the FSF Emacs copyright assignment process. Would you be willing to do that?

Thank you so much for your amazing packages and work. All the best from Portugal

Thanks for the kind words and your contribution.

@alphapapa alphapapa self-assigned this Mar 28, 2024
@alphapapa alphapapa added enhancement New feature or request author needs FSF CA labels Mar 28, 2024
@alphapapa alphapapa added this to the 0.8 milestone Mar 28, 2024
@fpvmorais
Copy link
Author

Sure I would be willing to complete the FSF Emacs copyright assignment process.
Can you please point me to where I could start that process?

Thanks

@alphapapa
Copy link
Owner

Sure, the way most people do it is to send a message to [email protected] and ask for the copyright assignment form to contribute to Emacs. Then the maintainers will send it to you with further instructions.

@alphapapa alphapapa modified the milestones: 0.8, v0.9 Apr 26, 2024
@alphapapa alphapapa modified the milestones: v0.9, v0.10 Jun 3, 2024
@fpvmorais
Copy link
Author

Hello

I finally have the document. How should we proceed now?

@alphapapa
Copy link
Owner

Sorry for the delay. Been quite busy. Feel free to ping me if I overlook something now and then.

Anyway, I am not allowed to take anyone's word for having completed the copyright assignment process. I need to receive confirmation from the FSF secretary or one of the Emacs maintainers. So you should either ask one of them to send confirmation to me, or tell me the name and email address you used when executing the document so I can ask them for confirmation.

@fpvmorais
Copy link
Author

fpvmorais commented Jun 30, 2024 via email

@alphapapa
Copy link
Owner

Hi Pedro,

Unfortunately, GitHub filters out the email addresses when you reply via email. So please either email me directly or contact the FSF secretary or Emacs maintainers and ask them to send confirmation to me.

Thanks.

@alphapapa
Copy link
Owner

I've received confirmation from the Emacs maintainers. I'll plan to merge this for v0.10.

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

Successfully merging this pull request may close these issues.

2 participants