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] Change description of skip_if_unavailable option #1358

Merged
merged 1 commit into from
May 15, 2019

Conversation

j-mracek
Copy link
Member

skip_if_unavailable option is now not only repo option but also option
an main config. option. The commit provides a description.

Additionally there is a change in default. Formally it was
skip_if_unavailable=true, but it was changed to False to provide a
better security to users. The commit also change the description
according to new default.

@j-mracek
Copy link
Member Author

Requires: rpm-software-management/libdnf#701

@lukash
Copy link
Contributor

lukash commented Mar 21, 2019

I think you wanted to use "formerly" instead of "formally"? Different meaning 😄

Also, maybe "... not only a repo option but also a main config option"? The end of that sentence looks like it's somehow mixed up.

@j-mracek
Copy link
Member Author

I hope that the commit message was improved.

skip_if_unavailable option is now not only a repo option but also option
of the main config. The commit provides a enhanced description.

Additionally there is a change in default. Formerly it was
skip_if_unavailable=true, but it was changed to False to provide a
better security to users. The commit also change the description
according to new default.
@jrohel jrohel self-assigned this May 15, 2019
@jrohel jrohel merged commit 1d97351 into rpm-software-management:master May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants