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

Link config management doc #3723

Merged
merged 2 commits into from
Feb 24, 2020
Merged

Link config management doc #3723

merged 2 commits into from
Feb 24, 2020

Conversation

balopat
Copy link
Contributor

@balopat balopat commented Feb 22, 2020

Fixes #2775.

@codecov
Copy link

codecov bot commented Feb 22, 2020

Codecov Report

Merging #3723 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted Files Coverage Δ
pkg/skaffold/sync/sync.go 72.25% <0%> (+0.32%) ⬆️

DEVELOPMENT.md Outdated
@@ -103,6 +103,8 @@ Once you've done this, merge or rebase your development branch with config chang

* Finally, before committing your final changes and opening your pull request, be sure and run `make test` locally. This will regenerate the JSON schemas for the skaffold config with your new changes. Commit the resulting changes autogenerated by the scripts.

For more details behind the logic of config changes see [the Skaffold config management doc](https://docs.google.com/document/d/e/2PACX-1vRBN1vHjSGbs3ZmdvyzzhBn48NjeuP4OiTTG5mec_YrRzNDL63pFa1Xy1rgCUNiQXTjAE1QyvCr5yxX/pub).
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the document supposed to be public? I can't access it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is supposed to be public, huh, I'll update.

@balopat balopat requested a review from dgageot February 24, 2020 18:12
@dgageot dgageot merged commit 918b78e into master Feb 24, 2020
@briandealwis briandealwis deleted the config_management_doc branch April 14, 2020 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better config versioning management
3 participants