← Back to team overview

mvhub-dev team mailing list archive

Re: [Merge] lp:~saroun-sek/mvhub/sql_change into lp:mvhub

 

> small detail perhaps only of taste than absolute right/wrong
> 
> w/ change #424 , you don't need to include filenames.
> 
> The goal is to have the log entry make sense when the user does
> 
>    bzr log | more
> 
> If user wants filenames he can do
> 
>   bzr log -v | more
> 
> It also makes sense to do a seperate commit for each change
> 
> so you could reduce to:
> 
> bzr commit -m 'added sql statements from Reports.pm'  app-
> mvhub/conf/sql_select.lib
> 
>   bzr -m commit 'removed unused code'

Alright. I'll do that next time.  But the next few commits I have waiting to be pushed is like 424.
-- 
https://code.launchpad.net/~saroun-sek/mvhub/sql_change/+merge/28658
Your team MVHub Developers is subscribed to branch lp:mvhub.



References