Juju Lint

Merge ~aieri/juju-lint:bug/1958899 into juju-lint:master

  1. Git
  2. lp:~aieri/juju-lint
  3. bug/1958899
  4. Merge into master
Proposed by Andrea Ieri
Status: Merged
Approved by: James Troup
Approved revision: 729f400fd1f14075e71b462646920ff958046ada
Merged at revision: c831a29f1017991704a64b210315c1c16084bea0
Proposed branch: ~aieri/juju-lint:bug/1958899
Merge into: juju-lint:master
Diff against target: 16 lines (+3/-2)
1 file modified
README.md (+3/-2)
Reviewer Review Type Date Requested Status
Juju Lint maintainers Pending
Review via email: mp+414514@code.launchpad.net

Commit message

Mention the need to export charm defaults

Closes-Bug: 1958899

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision c831a29f1017991704a64b210315c1c16084bea0

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/README.md b/README.md
2index d7ef4c7..f418837 100644
3--- a/README.md
4+++ b/README.md
5@@ -9,9 +9,10 @@ To generate a status if you just want to audit placement:
6
7 juju status --format yaml > status.yaml
8
9-For auditing configuration, you would want:
10+For auditing configuration, you would want (you need charm defaults to avoid
11+false positives):
12
13- juju export-bundle > bundle.yaml
14+ juju export-bundle --include-charm-defaults > bundle.yaml
15
16 Then run `juju-lint` (using a rules file of `lint-rules.yaml`):
17

Subscribers

People subscribed via source and target branches