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

Update man page for ipa-adtrust-install by removing --no-msdcs option #237

Closed
wants to merge 1 commit into from

Conversation

pspacek
Copy link
Contributor

@pspacek pspacek commented Nov 14, 2016

@tkrizek tkrizek added the ack Pull Request approved, can be merged label Nov 14, 2016
@martbab martbab removed the ack Pull Request approved, can be merged label Nov 14, 2016
@martbab
Copy link
Contributor

martbab commented Nov 14, 2016

Please add the upstream ticket to the commit message.

@pspacek
Copy link
Contributor Author

pspacek commented Nov 14, 2016

Here you go.

@martbab martbab added the ack Pull Request approved, can be merged label Nov 14, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Nov 14, 2016
@MartinBasti
Copy link
Contributor

@pspacek
Copy link
Contributor Author

pspacek commented Nov 15, 2016

Given the code change went to ipa-4-4 branch, I would merge it to ipa-4-4 as well. It does not make sense to keep it only in 4.5 and have incorrect documentation in 4.4.

@pspacek pspacek reopened this Nov 15, 2016
@MartinBasti
Copy link
Contributor

So please re-triage it today, yesterday was only 4.5 milestone agreed

@MartinBasti
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
4 participants