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

docs: document azure keyring usage #1913

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

tdejager
Copy link
Contributor

Added some short documentation regarding Azure authentication. Could @jslorrma, verify its correctness if you have the chance? Also pulling in @PanTheDev, because I know you wanted to add documentation surrounding this. We can modify/edit as you guys see fit!

@jslorrma
Copy link
Contributor

LGTM!

FYI: I have added the keyrings.artifacts package to conda-forge staged-recipes and opened a PR to add my package to conda-forge feedstocks. The recipe has an entry-point for the keyring script, so that when the package is available from conda-forge a global installation with pixi global install keyrings.artifacts should be possible, that automatically adds keyring script to ~/.pixi/bin

@tdejager
Copy link
Contributor Author

LGTM!

FYI: I have added the keyrings.artifacts package to conda-forge staged-recipes and opened a PR to add my package to conda-forge feedstocks. The recipe has an entry-point for the keyring script, so that when the package is available from conda-forge a global installation with pixi global install keyrings.artifacts should be possible, that automatically adds keyring script to ~/.pixi/bin

Nice!

@tdejager tdejager changed the title doc: document azure keyring usage docs: document azure keyring usage Aug 27, 2024
@tdejager
Copy link
Contributor Author

@ruben-arts please review

@ruben-arts ruben-arts merged commit ff86874 into prefix-dev:main Aug 28, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants