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 REPL quickstart subsection #676

Merged
merged 1 commit into from
Jun 10, 2022
Merged

Add REPL quickstart subsection #676

merged 1 commit into from
Jun 10, 2022

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented May 18, 2022

This PR adds a short section to the readme describing how to use the library directly in a REPL.

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

7 similar comments
@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

@jujubot
Copy link
Contributor

jujubot commented May 18, 2022

Can one of the admins verify this patch?

Copy link
Contributor

@juanmanuel-tirado juanmanuel-tirado left a comment

Choose a reason for hiding this comment

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

LG2M
Thanks for the contribution.

@cderici
Copy link
Contributor

cderici commented May 31, 2022

/merge

@cderici
Copy link
Contributor

cderici commented May 31, 2022

/build

1 similar comment
@cderici
Copy link
Contributor

cderici commented May 31, 2022

/build

@cderici
Copy link
Contributor

cderici commented Jun 9, 2022

/merge

@cderici
Copy link
Contributor

cderici commented Jun 9, 2022

/build

@cderici
Copy link
Contributor

cderici commented Jun 9, 2022

/merge

@cderici
Copy link
Contributor

cderici commented Jun 9, 2022

@sed-i I believe the CI here is stuck with one of the weird CI issues we were having lately and requires one of the recent fixes for it, simple rebase on the main branch should make it mergable 👍

@sed-i
Copy link
Contributor Author

sed-i commented Jun 10, 2022

Rebased. Thanks @cderici!

@cderici
Copy link
Contributor

cderici commented Jun 10, 2022

/build

1 similar comment
@cderici
Copy link
Contributor

cderici commented Jun 10, 2022

/build

@cderici
Copy link
Contributor

cderici commented Jun 10, 2022

/merge

@jujubot jujubot merged commit bd9490d into juju:master Jun 10, 2022
jujubot added a commit that referenced this pull request Jul 12, 2022
#688

## What's Changed
* Add REPL quickstart subsection by @sed-i in #676
* Revision of test onos.charm by @juanmanuel-tirado in #686
* [JUJU-1353] Parse assume directives. by @juanmanuel-tirado in #685
* Replace deprecated juju.loop() calls from examples and documentation by @ittner in #687
* Fixed the bundle run when the channel is None by @oEscal in #664

[JUJU-1353]: https://warthogs.atlassian.net/browse/JUJU-1353?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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.

4 participants