Skip to content

Commit

Permalink
add to-do comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Reuven committed Jun 21, 2022
1 parent 48c42f2 commit 66a8e66
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions diff/callbacks_diff.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ func (diff *CallbacksDiff) removeNonBreaking() {
return
}

// TODO: check breaking conditions
diff.Added = nil
}

Expand Down
1 change: 1 addition & 0 deletions diff/content_diff.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ func (diff *ContentDiff) removeNonBreaking() {
return
}

// TODO: check breaking conditions
diff.MediaTypeAdded = nil
}

Expand Down
1 change: 1 addition & 0 deletions diff/encodings_diff.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ func (diff *EncodingsDiff) removeNonBreaking() {
return
}

// TODO: check breaking conditions
diff.Added = nil
}

Expand Down
1 change: 1 addition & 0 deletions diff/headers_diff.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ func (headersDiff *HeadersDiff) removeNonBreaking() {
return
}

// TODO: check if we need to consider header.Required flags
headersDiff.Added = nil
}

Expand Down
1 change: 1 addition & 0 deletions diff/parameters_diff.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ func (parametersDiff *ParametersDiff) removeNonBreaking(params2 openapi3.Paramet
return
}

// TODO: check vs. SchemaDiff.removeChangedButNonRequiredProperties
parametersDiff.removeAddedButNonRequired(params2)
}

Expand Down

0 comments on commit 66a8e66

Please sign in to comment.