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

Add dbg package #51

Merged
merged 1 commit into from Jan 8, 2019

Conversation

Projects
None yet
1 participant
@Carpetsmoker
Copy link
Member

Carpetsmoker commented Jan 8, 2019

Often written ad-hoc version of this; sometimes when debugging code you
print out data:

fmt.Println("Append", tbl)

But not always clear where that's being called from; so we can now use:

fmt.Println("Append", tbl, dbg.Loc(1))

And get:

Append hub_installations_apps installation.go:492
Append hub_installations_apps installation.go:36

Which is useful :-)

I kep the package and function name short so it's quick to type.

Pedro: "it's 2019 and use a debugger!" Sorry, I'm not very hip :-(

Add dbg package
Often written ad-hoc version of this; sometimes when debugging code you
print out data:

	fmt.Println("Append", tbl)

But not always clear where that's being called from; so we can now use:

	fmt.Println("Append", tbl, dbg.Loc(1))

And get:

    Append hub_installations_apps installation.go:492
    Append hub_installations_apps installation.go:36

Which is useful :-)

I kep the package and function name short so it's quick to type.

Pedro: "it's 2019 and use a debugger!" Sorry, I'm not very hip :-(
@codecov

This comment has been minimized.

Copy link

codecov bot commented Jan 8, 2019

Codecov Report

Merging #51 into master will decrease coverage by 0.89%.
The diff coverage is 0%.

Impacted Files Coverage Δ
dbg/dbg.go 0% <0%> (ø)

@Carpetsmoker Carpetsmoker merged commit 9d322be into master Jan 8, 2019

2 of 4 checks passed

codecov/patch 0% of diff hit (target 78.37%)
Details
codecov/project 77.48% (-0.9%) compared to 702eaea
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@Carpetsmoker Carpetsmoker deleted the dbg branch Jan 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment