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

Initial commit to go mod #180

Merged
merged 18 commits into from
May 4, 2020
Merged

Initial commit to go mod #180

merged 18 commits into from
May 4, 2020

Conversation

notzippy
Copy link
Contributor

@notzippy notzippy commented Apr 10, 2020

This is a WIP for upgrading the command line tool to use go modules by default
What is done

  • Create new project, you can specify the package name as well.
  • go.mod files are created in the base of the package source code and can be found by the revel run command
  • The function ProcessSource old new has been started to be rewritten to use the go module friendly reflection tool at golang.org/x/tools/go/package.

ToDo

  • Follow the code for the old and add that into the new.

The goal is to completely rewrite the ProcessSource function, and we will eventually drop the old implementation

notzippy and others added 8 commits April 26, 2020 22:28
Added processor object to code
Verified compile errors appearing
Signed-off-by: notzippy@gmail.com
…late the SourceInfo object the same as before
Added packagepathmap to the SourceInfo, this in turn allows the RevelCLI app command to pass the source paths directly to Revel directly
Added default to build to be "target" of the current folder
Renamed source processor
Removed go/build reference in clean
Ran through testing individually for vendored Revel applications
notzippy@gmail.com added 3 commits April 26, 2020 23:00
remaps the go mod command to use the develop branch.
Modified to use fsnotify directlyUpdated travis to not use go deps
notzippy@gmail.com added 2 commits April 29, 2020 17:05
Added a gomod-flags parameter which allows you to run go mod commands on the go.mod file before the build is performed. This allows for development environments.
Added verbose flag so we can see what is occurring,
Removed checkout for revel, not needed anymore
Copy link
Contributor

@ptman ptman left a comment

Choose a reason for hiding this comment

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

some comments

go.mod Outdated Show resolved Hide resolved
harness/app.go Show resolved Hide resolved
harness/build.go Outdated Show resolved Hide resolved
harness/build.go Show resolved Hide resolved
@ptman ptman mentioned this pull request Apr 30, 2020
Updated NotVendored flag
Updated travis matrix
Updated build log
@notzippy notzippy force-pushed the go-mod branch 2 times, most recently from 7ac9f88 to b089001 Compare May 2, 2020 19:14
Modified GOPATH to not modify build with go.mod
Updated go.mod to version 1.12
Updated harness to setup listener before killing process
Updated notvendored flag to --no-vendor
Updated command_config to ensure no-vendor can be build
Added additional checks in source path lookup
@notzippy notzippy merged commit 60b88a4 into revel:develop May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants