Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: fix skipping disabled observability charms #1475

Merged
merged 5 commits into from
Nov 28, 2024

Conversation

dimaqq
Copy link
Contributor

@dimaqq dimaqq commented Nov 28, 2024

Took 5 attempts, but I got the syntax right after all.

Copy link
Contributor

@tonyandrewmeyer tonyandrewmeyer left a comment

Choose a reason for hiding this comment

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

Assuming correct. Thanks!

Copy link
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.

I see no reason to think that the syntax is incorrect this time

@dimaqq dimaqq merged commit e573f8f into canonical:main Nov 28, 2024
32 checks passed
@dimaqq dimaqq deleted the chore-fix-ci-matrix branch November 28, 2024 06:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants