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

[JUJU-4204] Model.name on 2.9 #902

Merged
merged 2 commits into from Jul 12, 2023
Merged

[JUJU-4204] Model.name on 2.9 #902

merged 2 commits into from Jul 12, 2023

Conversation

cderici
Copy link
Member

@cderici cderici commented Jul 10, 2023

Description

Backports model.name from HEAD into 2.9. Gets this particular change from 55ea0e2 adds the name property into a Model object. Couldn't directly cherry pick because of the other stuff in the commit.

Fixes #893

QA Steps

Also backports the integration test as well, so:

tox -e integration -- tests/integration/test_model.py::test_model_name

Gets this particular change from
55ea0e2 adds the name @Property into
a Model object. Couldn't directly cherry pick because of the other
stuff in the commit.
@cderici cderici added the 2.9 label Jul 10, 2023
@cderici cderici requested a review from jack-w-shaw July 10, 2023 21:36
@cderici
Copy link
Member Author

cderici commented Jul 12, 2023

/merge

@jujubot jujubot merged commit 4c44e9b into juju:2.9 Jul 12, 2023
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants