-
Notifications
You must be signed in to change notification settings - Fork 51
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
Document 'tmt_plan' and 'tf_post_install_script' #491
Comments
ah, I just realized Rodolfo already resolved this in #489 pity he didn't link the docs PR with the p-s PR |
Shouldn't we document everything on https://packit.dev/docs/configuration/, too? In order to have a single page where everything can be found in a single place. |
oh, I missed it's in the testing-farm document and not the configuration; agreed! |
There are only links in configuration.md pointing to testing-farm. Do we want to keep this trend or actually have a single page with all the configuration values? |
I'm not sure, but so far I found it handy to be able to open the configuration page and look up any configuration option. As a user I would expect something like this, or if there are more options, then maybe multiple pages, but then it should be clear which page covers what. Right now I don't think it's self-explanatory that the "Testing Farm" page also documents configuration, or that the Configuration page doesn't document options related to Testing Farm. |
+1 I also |
Related to #467 |
Closing. #467 should fix this. |
Was introduced in packit/packit#1619.
The text was updated successfully, but these errors were encountered: