Closed lsetiawan closed 6 years ago
@lsetiawan Thanks for taking these steps. I won't be able to review this until late today or tomorrow (it's 12 files changed, and it's more complicated than usual!).
In the meantime, in the warning text, the word "deprecated" is misspelled everywhere as "depricated". You can go ahead and fix that.
@lsetiawan I have a question for you, that may be a recommendation. I can see that you edited in place 5 modules in the ODM2
directory we're deprecating, and copied/created versions of those modules (plus services/__init__.py
) in the new tree hierarchy that starts at the base path level. But as far as I can tell the relocated files (or copies) were created as "new" files, meaning -- and this is the question -- that they won't retain their git
commit history; all the history will remain with the files in the old, deprecated ODM2
directory, which will eventually be removed.
Is that interpretation correct? If so, can you re-do this PR to implement exactly the same changes you've submitted, but done in such a way that you maintain the full git
history in the new files? ie, we'd still be able to do a git diff
to see what's changed relative to any commit of our choice.
Notes to self (my progress status; @lsetiawan, ignore this):
ODM2
directory/hierarchy to the base package level; 5 of those are in the services
directory, and the other one is models.py
ODMconnection.py
was already at the base package level, so it did not need to be moved. Only one simple change was made, to an import
statement, and it looks OkODM2
folderIs that interpretation correct? If so, can you re-do this PR to implement exactly the same changes you've submitted, but done in such a way that you maintain the full git history in the new files? ie, we'd still be able to do a git diff to see what's changed relative to any commit of our choice.
Your interpretation is correct... I'm not sure how to do that in git.
Is that interpretation correct? If so, can you re-do this PR to implement exactly the same changes you've submitted, but done in such a way that you maintain the full git history in the new files? ie, we'd still be able to do a git diff to see what's changed relative to any commit of our choice.
Your interpretation is correct... I'm not sure how to do that in git.
Ok. Tell me if I'm wrong, but this sequence (order) is what I have in mind:
git mv
to move all the files you're moving from ODM2
to the base pathODM2
directory (it looks like git
has no git cp
command?)git
history will stay with the modules that will remain after the ODM2
is removed, not with the modules that will disappearOkay. I will try that on Friday. Thanks.
Please test this branch anyhow to make sure that the new and old functions actually work if you have some time. Thanks.
Okay. I will try that on Friday. Thanks.
So, does my suggestion make sense to you?? You're the git expert here, not me!
Please test this branch anyhow to make sure that the new and old functions actually work if you have some time. Thanks.
I'll try, but no promises ...
So, does my suggestion make sense to you?? You're the git expert here, not me!
Yepp. It makes sense, just never done it.
@lsetiawan are you planning to implement today the PR changes I recommended?
Just trying to see what to expect today. I have a meeting at 2pm at Suzzallo.
@emiliom I am about to do a PR, but it seems like, that didn't workout... The change history is still exactly the same.. hmmm.
Hmm
Closing this PR, replaced by #147
NOT READY TO BE MERGEDOverview
This is a test of the deprecation for
ODM2
module folder.