Skip to content

Add a diff sub command and more parameter#43

Merged
olblak merged 5 commits into
masterfrom
develop
May 3, 2020
Merged

Add a diff sub command and more parameter#43
olblak merged 5 commits into
masterfrom
develop

Conversation

@olblak

@olblak olblak commented May 3, 2020

Copy link
Copy Markdown
Member
  • Add parameters [--commit, --push, --clean], in order to not commit/push/clean if needed
  • Add diff subcommand to show what would change
  • Improve documentation

@olblak olblak added the enhancement New feature or request label May 3, 2020
@olblak olblak merged commit 2a5742f into master May 3, 2020
olblak added a commit that referenced this pull request Feb 1, 2023
* feat(yaml): implement spec.files for kind: yaml

Follow-up of #490
Ref #435

* Add e2e test for yaml (#43)

* Apply suggestions from code review

Co-authored-by: Olivier Vernin <olivier@vernin.me>

* chore: move test to e2e success folder

* wip: first stab at files for condition too

* Revert "wip: first stab at files for condition too"

This reverts commit 957f21e.

* restore comment, will be done in #729

* chore: replace deprecated sourceID field by sourceid

* fix: e2e tests

* breaking?: no error if the target content hasn't changed

* Remove broken test file

Signed-off-by: Olblak <me@olblak.com>

* patch from main (with #846 included and adapted)

* fix tests

* Add missing jsonschema annotations

Co-authored-by: Olivier Vernin <olivier@vernin.me>

* fix: remove files field from expected autodiscovery tests results
Omited if empty since previous commit

* cleanup: remove non mandatory empty files field from chart examples

* chore: more complete debug log with initial filePath

* chore: add debug logs

* Update pkg/plugins/resources/yaml/main.go

* chore: add missing Test_TargetFromSCM test

* Update pkg/plugins/resources/yaml/source.go

Co-authored-by: Hervé Le Meur <91831478+lemeurherve@users.noreply.github.com>

* Update pkg/plugins/resources/yaml/main.go

Co-authored-by: Olivier Vernin <olivier@vernin.me>

* allow multiples files for 'condition'

* update example with a condition checking multiple files

* remove TODOs

* remove unrelated changes

* replace example by e2e test

* correct check for s.file

* Update pkg/plugins/resources/yaml/main.go

---------

Signed-off-by: Olblak <me@olblak.com>
Co-authored-by: Olivier Vernin <olivier@vernin.me>
Co-authored-by: Olblak <me@olblak.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant