close
Skip to content

docs: update referenced examples for managing interfaces#2068

Merged
dwilding merged 2 commits intocanonical:mainfrom
dwilding:remove-minimal-interface-example
Sep 30, 2025
Merged

docs: update referenced examples for managing interfaces#2068
dwilding merged 2 commits intocanonical:mainfrom
dwilding:remove-minimal-interface-example

Conversation

@dwilding
Copy link
Copy Markdown
Contributor

@dwilding dwilding commented Sep 29, 2025

In "How to manage interfaces" we have some links to a repo that demonstrated the code shown in the doc. But over time some of the links have broken. In a team discussion we decided to remove the references and keep the links to examples that are in active use.

This PR updates the references. In the future, we may need to make more substantial changes as interfaces move to the charmlibs repo.

Preview doc

Drive-by: pre-commit cleaned up some whitespace. I suggest reviewing with whitespace hidden.

@dwilding
Copy link
Copy Markdown
Contributor Author

Not a high priority to review - thanks

@dwilding dwilding marked this pull request as ready for review September 29, 2025 11:53
Copy link
Copy Markdown
Contributor

@james-garner-canonical james-garner-canonical left a comment

Choose a reason for hiding this comment

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

Looks good to me, nice to clean these up; and a good reminder that this doc will need to be updated as the interface testing work progresses.

Comment thread docs/howto/manage-interfaces.md Outdated
Copy link
Copy Markdown
Collaborator

@benhoyt benhoyt left a comment

Choose a reason for hiding this comment

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

Thanks!

@dwilding dwilding merged commit 2666e37 into canonical:main Sep 30, 2025
41 checks passed
@dwilding dwilding deleted the remove-minimal-interface-example branch September 30, 2025 01:51
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.

3 participants