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

Add aah migrate command to fix/upgrade application codebase to latest aah version #116

Closed
2 tasks done
jeevatkm opened this issue Sep 9, 2017 · 6 comments
Closed
2 tasks done
Assignees
Labels
Projects

Comments

@jeevatkm
Copy link
Member

jeevatkm commented Sep 9, 2017

The goal is to bring aah migrate command to easy the upgrade or fix the aah framework breaking changes and deprecated elements in .go files and basic template file support.

Limitation:
To begin with migrate command does not support config files; it should be done manually, later I will add it.

  • Implementation
  • Documentation
@jeevatkm jeevatkm added cli-tool aah CLI tool feature labels Sep 9, 2017
@jeevatkm jeevatkm added this to Backlog in aah Roadmap Sep 9, 2017
@julienkosinski
Copy link

@jeevatkm About config file, when you'll add it update template directly with new config and comments (because config file are always really well commented) with an auto-merging process (like git could do with pulling commits).

For resolving conflicts some of them could modify the project with find/replace. I'm thinking of something like obvious typo change in API...
For all the complexe changes to make I'm thinking of logging in CLI each modifications and actions required and modify the file in which an action is required adding comments in the problematic line to help resolve the problem (maybe the same comments logged in CLI)

@jeevatkm
Copy link
Member Author

@julienkosinski That's good suggestion. I agree, logging information via CLI would be helpful.

@julienkosinski
Copy link

julienkosinski commented Sep 24, 2017

@jeevatkm Also, as config files updates regularly with new versions to update documentations, efforts could be made about this.

I know that (learn just now in fact), for example, modern javascript front-end scaffolding apps handle this by avoiding default config and providing config repo kept up-to-date with best-practices for each use cases. (See POI, here, here for example).

I suppose in this case the best way is to design the upgrade tool so that it merges config file but having a repo for config is not stupid because there's multiple config use cases to handle (and the created app needs to keep the info of what type of app it is and updates it accordingly, is it web or api, form or basic, etc).

What's your thought about this :)?

@jeevatkm
Copy link
Member Author

jeevatkm commented Oct 6, 2017

@julienkosinski I have read and went through your references of config templates idea. Looks interesting.

I will have to do analysis and homework on config template to see; it is feasible to accommodate in aah and Go. Will get back to you.

I would like to do proper homework before touching migrate command implementation.

@jeevatkm jeevatkm self-assigned this May 25, 2018
@jeevatkm jeevatkm added this to the v0.11.0 Milestone milestone May 25, 2018
@jeevatkm jeevatkm moved this from Backlog to v0.11.0 - In Progress in aah Roadmap May 25, 2018
jeevatkm added a commit to go-aah/tools that referenced this issue May 26, 2018
@jeevatkm
Copy link
Member Author

Implementation is done (I would call it beta to begin with; gradually it will get more mature).

Have to work in documentation 😄

jeevatkm added a commit to go-aah/docs that referenced this issue May 27, 2018
@jeevatkm
Copy link
Member Author

Its finally done, thank you for patience.

aah Roadmap automation moved this from v0.11.0 - In Progress to v0.11.0 - Completed May 27, 2018
@jeevatkm jeevatkm moved this from v0.11.0 - Completed to Released to Audience in aah Roadmap Jul 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
aah Roadmap
  
Released to Audience
Development

No branches or pull requests

2 participants