Skip to content

Remove deprecated functions #89

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.

Sign up for GitHub

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

Merged
RincewindsHat merged 3 commits into from
Jun 26, 2023
Merged

Remove deprecated functions #89

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f074560
Add deprecation notice to http/mock module
martialblog Jun 26, 2023
bf3da37
Remove deprecated Exit() function
martialblog Jun 26, 2023
6dc9a61
Remove deprecated Add Functions
martialblog Jun 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions exit.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,13 +48,6 @@ func ExitRaw(rc int, output ...string) {
BaseExit(rc)
}

// Exit prints the plugin output and exits the program
//
// Deprecated, please use Exitf or ExitRaw.
func Exit(rc int, output string, args ...interface{}) {
Exitf(rc, output, args...)
}

func BaseExit(rc int) {
if AllowExit {
os.Exit(rc)
Expand Down
1 change: 1 addition & 0 deletions http/mock/mock.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,5 @@
// Features:
// - RegisterQueryMapResponder form query based based on a QueryMap
// - ActivateRecorder to record HTTP requests during the development of tests and examples
// Deprecated: Will be removed in a future version
package checkhttpmock
20 changes: 0 additions & 20 deletions result/overall.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,26 +42,6 @@ func (s *PartialResult) String() string {
return fmt.Sprintf("[%s] %s", check.StatusText(s.GetStatus()), s.Output)
}

// Deprecated: Will be removed in a future version, use Add() instead
func (o *Overall) AddOK(output string) {
o.Add(check.OK, output)
}

// Deprecated: Will be removed in a future version, use Add() instead
func (o *Overall) AddWarning(output string) {
o.Add(check.Warning, output)
}

// Deprecated: Will be removed in a future version, use Add() instead
func (o *Overall) AddCritical(output string) {
o.Add(check.Critical, output)
}

// Deprecated: Will be removed in a future version, use Add() instead
func (o *Overall) AddUnknown(output string) {
o.Add(check.Unknown, output)
}

// Add State explicitely
// Hint: This will set stateSetExplicitely to true
func (o *Overall) Add(state int, output string) {
Expand Down
2 changes: 1 addition & 1 deletion result/overall_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ func ExampleOverall_withSubchecks() {
subcheck.SetState(check.OK)

overall.AddSubcheck(subcheck)
overall.AddOK("bla")
overall.Add(0, "bla")

fmt.Println(overall.GetOutput())
// Output:
Expand Down