Skip to content

Conversation

@odanoburu
Copy link

hi! I'm using this project (thanks!) and have been writing a few patches. what do you think of them? should I keep sending them your way? (I should probably update the documentation too)

add variables for Neo4j username and password

and use them to log in. We could pass these using the cli-arguments variable, but then the password would be stored as plaintext.

accumulate comint process input

effectively allows for easier multiline cypher queries

use variable for cypher-shell buffer name

and use them to log in

* accumulate comint process input

effectively allows for easier multiline cypher queries

* use variable for cypher-shell buffer name
  users can customize all arguments to the =cypher-shell= program by using the
  ‘n4js-cli-other-arguments’ variable; however we want there to be an easier way to
  customize the most common options (address, username, and password). now there
  is.
@odanoburu
Copy link
Author

odanoburu commented Aug 21, 2020

hi, this is just a reminder in case you forgot about this!

if you didn't forget but don't have time and/or interest for this, would you mind making me a maintainer or transferring the repo?

* make cypher-mode dependency optional

* revert naming of n4js-cli-arguments

to not break backwards-compatibility
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.

1 participant