Code review comment for lp:~shawn111/checkbox/image_info
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
More inline comments below. Please ping me again after addressing all of the comments.
One more thing, to make it explicit: please define the version of Ubuntu that this script aims to support. This is important for things like udisks API and transition processes. Define your requirements and we'll see what to do about that.
Lastly. I'm somewhat tempted to move all of the DELL-specific stuff into a DELL provider though this can be done later.