Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: remove the description of deleted options #10741

Merged
merged 1 commit into from Aug 19, 2016
Merged

doc: remove the description of deleted options #10741

merged 1 commit into from Aug 19, 2016

Conversation

minshenglin
Copy link
Contributor

@minshenglin minshenglin commented Aug 16, 2016

@smithfarm
Copy link
Contributor

added http://tracker.ceph.com/issues/17038 to the PR description

@minshenglin
Copy link
Contributor Author

minshenglin commented Aug 16, 2016

@smithfarm done. Thanks for your help!

@tchaikov
Copy link
Contributor

@minshenglin could you put this way,

Fixes: http://tracker.ceph.com/issues/17041
Fixes: http://tracker.ceph.com/issues/17038

sorry for being picky.

@tchaikov
Copy link
Contributor

@minshenglin BTW, i guess we can just close http://tracker.ceph.com/issues/17038 as "won't fix".

@minshenglin
Copy link
Contributor Author

@tchaikov Do you mean we can close issue 17038 as "won't fix" and remove "Fixes: http://tracker.ceph.com/issues/17038" in commit message?

@smithfarm
Copy link
Contributor

@minshenglin I marked it WON'T FIX. Now you can remove "Fixes: http://tracker.ceph.com/issues/17038" in commit message.

Fixes: http://tracker.ceph.com/issues/17041

Signed-off-by: MinSheng Lin <minsheng.l@inwinstack.com>
@minshenglin
Copy link
Contributor Author

@smithfarm done.

@tchaikov
Copy link
Contributor

lgtm.

@tchaikov tchaikov self-assigned this Aug 16, 2016
@tchaikov tchaikov merged commit 2bc985f into ceph:master Aug 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants