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

Support Go modules #65

Merged
merged 2 commits into from
May 7, 2020
Merged

Support Go modules #65

merged 2 commits into from
May 7, 2020

Conversation

d-kuro
Copy link
Contributor

@d-kuro d-kuro commented May 7, 2020

By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Add support for Go Modules.

$ go mod init github.com/auth0/go-jwt-middleware
go: creating new go.mod: module github.com/auth0/go-jwt-middleware

$ go mod tidy
go: finding module for package github.com/dgrijalva/jwt-go
go: finding module for package github.com/go-martini/martini
go: finding module for package github.com/urfave/negroni
go: finding module for package github.com/gorilla/mux
go: finding module for package github.com/smartystreets/goconvey/convey
go: downloading github.com/dgrijalva/jwt-go v1.0.2
go: downloading github.com/smartystreets/goconvey v1.6.4
go: downloading github.com/go-martini/martini v0.0.0-20170121215854-22fa46961aab
go: found github.com/dgrijalva/jwt-go in github.com/dgrijalva/jwt-go v3.2.0+incompatible
go: found github.com/go-martini/martini in github.com/go-martini/martini v0.0.0-20170121215854-22fa46961aab
go: found github.com/gorilla/mux in github.com/gorilla/mux v1.7.4
go: found github.com/urfave/negroni in github.com/urfave/negroni v1.0.0
go: found github.com/smartystreets/goconvey/convey in github.com/smartystreets/goconvey v1.6.4
go: finding module for package github.com/codegangsta/inject
go: downloading github.com/codegangsta/inject v0.0.0-20150114235600-33e0aa1cb7c0
go: found github.com/codegangsta/inject in github.com/codegangsta/inject v0.0.0-20150114235600-33e0aa1cb7c0

$ go get -u all
go: github.com/smartystreets/assertions upgrade => v1.1.0
go: github.com/gopherjs/gopherjs upgrade => v0.0.0-20200217142428-fce0ec30dd00
go: downloading github.com/smartystreets/assertions v1.1.0
go: downloading github.com/gopherjs/gopherjs v0.0.0-20200217142428-fce0ec30dd00

References

None

Testing

No changes

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used, if not master

@copland
Copy link

copland commented May 7, 2020

@d-kuro Thanks for the pull request! Will take a look at this shortly.

Copy link

@copland copland left a comment

Choose a reason for hiding this comment

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

LGTM

@copland copland merged commit d30d7b9 into auth0:master May 7, 2020
@d-kuro d-kuro deleted the feature/goimports branch May 7, 2020 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants