Skip to content

Conversation

@ejmr
Copy link

@ejmr ejmr commented Sep 23, 2018

This patch provides an interactive function which will prompt the
user for the name of one of Phpactor's RPC commands, and will proceed
to open the user's browser to the documentation on that command.
For the documentation we use the Phpactor GitHub page because
it's Markdown document of RPC commands contains hyperlinks to each
individual command.

In the future we should probably restrict the possible inputs to
only those command names which are valid. However, that assumes that
phpactor.el has a built-in list of valid commands. If not, or if
we decide we do not wish to maintain such a list, then we should not
bother with this addition.

Finally, this patch does not update the README with any mention of
the new function as I could not think of a succinct way to describe it.

Signed-off-by: Eric James Michael Ritz ejmr@no.current.address

This patch provides an interactive function which will prompt the
user for the name of one of Phpactor's RPC commands, and will proceed
to open the user's browser to the documentation on that command.
For the documentation we use the Phpactor GitHub page because
it's Markdown document of RPC commands contains hyperlinks to each
individual command.

In the future we should probably restrict the possible inputs to
only those command names which are valid.  However, that assumes that
`phpactor.el` has a built-in list of valid commands.  If not, or if
we decide we do not wish to maintain such a list, then we should not
bother with this addition.

Finally, this patch does not update the README with any mention of
the new function as I could not think of a succinct way to describe it.

Signed-off-by: Eric James Michael Ritz <ejmr@no.current.address>
@ejmr ejmr added the enhancement New feature or request label Sep 23, 2018
@ejmr
Copy link
Author

ejmr commented Sep 23, 2018

If you want to test this out, M-x phpactor-open-rpc-documentation and then enter move_class, etc.

@zonuexe zonuexe merged commit 8449e10 into master Sep 23, 2018
@zonuexe
Copy link
Member

zonuexe commented Sep 23, 2018

@ejmr Thanks!

@zonuexe zonuexe deleted the lookup-rpc-documentation branch March 25, 2020 12:20
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.

3 participants