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

Module credspropagation would benefit from defining keys as constants #763

Open
s3rj1k opened this issue Dec 10, 2024 · 1 comment
Open

Comments

@s3rj1k
Copy link
Contributor

s3rj1k commented Dec 10, 2024

In credspropagation, for different/supported providers would be nice to have string fields in maps as constants with appropriate documentation links (like provider schema docs) for each parameter where available.

@zerospiel
Copy link
Member

I believe it might be enough to leave doc links within each symbol's body, without making each key an external constant (most of them are self-describing), it multiplies the number of lines x2, as well as attempts to put links on each from the params having the only link to the corresponding documentation page. @a13x5 WDYT?

@s3rj1k s3rj1k changed the title Module credspropagation would benifit from defining keys as constants Module credspropagation would benefit from defining keys as constants Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

2 participants