search

cloudmesh / cloudmesh-rest

experimental REST services with eve
Other
0 stars 8 forks source link

Several commits converged in this Pull request #16

Closed campfire29 closed 7 years ago

campfire29 commented 7 years ago

Update to the Report Production Section: The first two paragraphs were updated with additional text (some from other volumes for consistency). I integrated the existing text into the new first paragraph. The "TBD" and the paragraph following the "TBD" were left as-is.

The second commit in this pull request is for updates to Section 1.1 only.

campfire29 commented 7 years ago

My apologies. It looks like I added two commits to this pull request. I intended to make them separate pull requests to make your review easier. Please let me know if this causes issues and I can redo them.

campfire29 commented 7 years ago

Please take note of the removal of the Contribution appendix. Please give me a call if you have any questions about the reasoning. Thanks.

laszewsk commented 7 years ago

you are makeing too many changes in one pull request, naturally i have to preserve the contribution section somewhere else so i remember how to do it, deleting will just lead to losing the information and me having to reinvent it. I think in these cases it would be better to just put a TODO in there or file an issue

laszewsk commented 7 years ago

THis pull request contains several important sections that are suggested to be removed. Others add information. Thus we can not approve this pull request. It is therefore rejected and has to be done with individual pull requests