Skip to content

Commit

Permalink
Merge pull request #4165 from dgageot/skaffold-yaml-doc-no-double-pf
Browse files Browse the repository at this point in the history
Don’t duplicate the definition of `portForward`, under the `profiles` section.
  • Loading branch information
tejal29 authored May 13, 2020
2 parents d7d0e87 + 2a5a7b9 commit b35b0f0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/content/en/docs/references/yaml/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,8 @@ function* template(definitions, parentDefinition, ref, ident, parent) {
// Description
const desc = definition['x-intellij-html-description'];

// Special case for profiles
if ((name === 'Profile') && (key === 'build' || key === 'test' || key === 'deploy')) {
// Don't duplicate definitions of top level sections such as build, test, deploy and portForward.
if ((name === 'Profile') && definitions['SkaffoldConfig'].properties[key]) {
value = '{}';
yield html`
<tr>
Expand Down

0 comments on commit b35b0f0

Please sign in to comment.