An error occurred while fetching reviewers.
Resolve "Write System Maintenance guide"
All threads resolved!
All threads resolved!
Initial SMG created and existing README has been updated to be directed more at users.
- SMG is purely for developers working on the data loader.
- README is purely for users of the data loader.
Closes #46 (closed)
Edited by Niall McDaid
Merge request reports
Activity
Filter activity
added Enhancement + 1 deleted label
requested review from @gregharris
assigned to @gregharris and @niall.mcdaid
- Resolved by Greg Harris
A couple of general comments. We should provide instructions on how to update the Setup.py
install_requires
field for new packages.We should provide a high level description of the "layers" of the application. eg:
- Command interface (Top level CLI related stuff,
wdbutil.py
file) - Command Processors (Action the commands, call service layer)
- Service Layer (Main business logic, including data mapping)
- OSDU and OS wrappers (Call OSDU, read files).
- Common (e.g. config, exceptions and utilities)
- Test code
I think there is an argument to divide the modules into a folder structure that follows this (we have a folder commands, that holds the command processors). What do you think?
wbdutil/ |--- wdbutil.py |--- commands/ |--- common/ |--- service/ |--- wrapper/
Edited by Greg Harris - Command interface (Top level CLI related stuff,
added 2 commits
added 22 commits
-
080f16a9...4eb463d1 - 8 commits from branch
main
- f9886614 - Minor corrections to README
- 153a45b8 - Initial sections of SMG
- 55013bc9 - Move dev setup info to SMG
- a2fb64d5 - Temporary file for tracking SMG progress
- b3e5c29a - Update refresh token
- 47089533 - auth tokens and merging policies
- 3071662b - Repository structure
- fbd032e0 - Minor updates to README
- d8d60cd6 - small rewording
- 47eb321f - Remove todo list
- 042f264a - Add info on setup.py
- eb6380b7 - SMG architecture section
- 2b8281a6 - automatic updates to setup dependencies
- 0480c826 - Update README - rebase changes
Toggle commit list-
080f16a9...4eb463d1 - 8 commits from branch
mentioned in commit eca7513d
added PriorityHigh label and removed 1 deleted label
Please register or sign in to reply