Skip to content

Add the option to read credentials from env variables#121

Merged
bedroesb merged 7 commits into
usegalaxy-eu:masterfrom
RZ9082:credentials
Mar 6, 2026
Merged

Add the option to read credentials from env variables#121
bedroesb merged 7 commits into
usegalaxy-eu:masterfrom
RZ9082:credentials

Conversation

@RZ9082

@RZ9082 RZ9082 commented Jan 29, 2026

Copy link
Copy Markdown
Member

No description provided.

Comment thread ena_upload/ena_upload.py Outdated
@RZ9082 RZ9082 marked this pull request as ready for review January 30, 2026 11:01
@RZ9082 RZ9082 moved this from Planned to Ready in Galaxy Tool Developers SIG Board Jan 30, 2026
@RZ9082 RZ9082 self-assigned this Jan 30, 2026
@RZ9082

RZ9082 commented Feb 6, 2026

Copy link
Copy Markdown
Member Author

Should we merge this one?

Comment thread ena_upload/ena_upload.py Outdated
@bernt-matthias

Copy link
Copy Markdown

Ping @bedroesb

@bedroesb

bedroesb commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

Hello, I will quickly do a test submission with this change. Son't forget to also update the documentation please.

@bedroesb

bedroesb commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

And thanks for the feature!

Comment thread ena_upload/ena_upload.py Outdated
@bedroesb

bedroesb commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

Just managed to successfully submit using both the secrets file and the env variables, so on that side, green light for me! (besides my previous comment)

@bedroesb bedroesb merged commit 44179e9 into usegalaxy-eu:master Mar 6, 2026
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