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

Jump to bottom

Move auto-generated files to a cache folder #199

Closed
radutopala opened this issue Jul 15, 2018 · 3 comments
Closed

Move auto-generated files to a cache folder #199

radutopala opened this issue Jul 15, 2018 · 3 comments
Assignees
@jeevatkm
Labels
cli-tool aah CLI tool enhancement
Projects
aah Roadmap
Milestone
v0.12.0 Milestone

Comments

@radutopala
Copy link

Store the auto-generated root/app/aah.go and root/app/aah_app_vfs.go maybe in root/app/cache/ or root/cache, just to be clear for the developer that these files shouldn't be editable and that they are auto-generated on each run or build.
@jeevatkm
Copy link
Member

@radutopala Agreed. Its good to differentiate auto-generated files.

I believe directory root/app/cache would be appropriate, since directory app is meant for Go source files.

@jeevatkm jeevatkm self-assigned this Jul 15, 2018
@jeevatkm jeevatkm added the cli-tool aah CLI tool label Jul 15, 2018
@jeevatkm jeevatkm added this to the v0.12.0 Milestone milestone Jul 15, 2018
@jeevatkm jeevatkm added this to v0.12.0 - Iteration in aah Roadmap via automation Jul 15, 2018
@jeevatkm jeevatkm moved this from v0.12.0 - Iteration to v0.12.0 - In Progress in aah Roadmap Oct 28, 2018
@jeevatkm
Copy link
Member

Going with directory name as generated at <app-base-directory>/app/generated. Since v0.12.0 release bring new module cache. Just on safer side and not to introduced confusion.

It will be imported on main file. Example -

 _ "<app-import-path>/app/generated"

This directory gets cleaned on every run, compile and build.

jeevatkm added a commit to go-aah/tools that referenced this issue Oct 28, 2018
@jeevatkm
go-aah/aah#198 go-aah/aah#199 integrated new console module into run …
9d9dcf1 
…and build flow and moved generated file into directory generated
@jeevatkm
Copy link
Member

Done 😄

aah Roadmap automation moved this from v0.12.0 - In Progress to v0.12.0 - Completed Oct 28, 2018
jeevatkm added a commit to go-aah/tools that referenced this issue Nov 2, 2018
@jeevatkm
go-aah/aah#198, go-aah/aah#199 integrated new console module into run…
865e513 
… and build flow and moved generated file into directory generated
@jeevatkm jeevatkm moved this from v0.12.0 - Completed to Released to Audience in aah Roadmap Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeevatkm jeevatkm

Labels
cli-tool aah CLI tool enhancement
Projects
aah Roadmap
  
Released to Audience
Milestone
v0.12.0 Milestone
Development

No branches or pull requests
2 participants
@radutopala @jeevatkm