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

Explain how to assign the response to a variable #55

Closed
wants to merge 1 commit into from

Conversation

amitaibu
Copy link
Contributor

@amitaibu amitaibu commented Apr 1, 2015

#54

@amitaibu
Copy link
Contributor Author

@micha rebased and ready for review

@AdrieanKhisbe
Copy link
Collaborator

Hello @amitaibu.
Just the fix the missing trailing ", with an ammend, and I'll cherry-pick it :)

@amitaibu
Copy link
Contributor Author

amitaibu commented Dec 6, 2015

@AdrieanKhisbe sorry, didn't understand your comment.

@@ -366,6 +366,8 @@ Since resty creates the REST verb functions in the shell, when using it from a s

Here the `-W` option was used when loading the script to prevent writing to the history file and an initial URI base was set at the same time. Then a JSON file was fetched, edited using [jsawk](http://github.com/micha/jsawk), and re-uploaded to the server.

To assign the response of resty to a variable you can you do for example: `VAR="$(GET /some/request)`. Note that the quote symbol (`"`) around the subcommand is necessary if the output contains spaces.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is a missing quote:

should be VAR="$(GET /some/request)"

@amitaibu
Copy link
Contributor Author

amitaibu commented Dec 6, 2015

Pushed a fix.

@AdrieanKhisbe
Copy link
Collaborator

can you squash your commit with a intreactive rebase
(in case, here is how to do it: http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)
:)

@amitaibu
Copy link
Contributor Author

amitaibu commented Dec 6, 2015

sure - wasn't sure you wanted it squashed. sec :)

@amitaibu
Copy link
Contributor Author

amitaibu commented Dec 6, 2015

Ready for review :)

Do I get bonus points for doing this, eventough I don't actually use resty? 😉

@AdrieanKhisbe
Copy link
Collaborator

Just cherry picked your commit. :)

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

Successfully merging this pull request may close these issues.

2 participants