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

Add pantry-sdk to sdk's #190

Closed
wants to merge 1 commit into from
Closed

Add pantry-sdk to sdk's #190

wants to merge 1 commit into from

Conversation

YieldRay
Copy link

use javascript's built-in fetch

@imRohan imRohan self-assigned this May 18, 2023
@imRohan imRohan added the Ready for Review Once you've squashed your commits, the PR is now ready for review! label May 18, 2023
@imRohan imRohan self-requested a review May 18, 2023 14:14
@imRohan imRohan assigned YieldRay and unassigned imRohan May 18, 2023
@imRohan
Copy link
Owner

imRohan commented May 18, 2023

Great work!

I'd request the PR (& commit message) be reworded to be more in line with other PRs like this.

Thanks!

@YieldRay YieldRay changed the title add pantry-sdk to sdk Add pantry-sdk to sdk's May 19, 2023
@YieldRay
Copy link
Author

Great work!

I'd request the PR (& commit message) be reworded to be more in line with other PRs like this.

Thanks!

Is this ok?

@imRohan
Copy link
Owner

imRohan commented Jun 6, 2023

Great work!
I'd request the PR (& commit message) be reworded to be more in line with other PRs like this.
Thanks!

Is this ok?

Yup all good.

Please squash your commits, and modify the commit message to match the changes you made to the PR message 👍

@YieldRay
Copy link
Author

YieldRay commented Jun 7, 2023

reworded

ok, reworded!

@imRohan
Copy link
Owner

imRohan commented Jun 7, 2023

Awesome! Looks like your branch needs to pull the latest changes from master.

Please pull those changes, and squash your commits to just 1. Then I will merge the PR :)

@YieldRay
Copy link
Author

YieldRay commented Jun 8, 2023

Force pushed to squash commits to only one, is it fine?

@YieldRay YieldRay closed this by deleting the head repository Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for Review Once you've squashed your commits, the PR is now ready for review!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants