Skip to content

Pyega3 - Move credentials to requirements #7594

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.

Sign up for GitHub

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

Jump to bottom
Open
RZ9082 wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Pyega3 - Move credentials to requirements #7594

RZ9082 wants to merge 15 commits into from

Conversation

@RZ9082
Copy link
Collaborator

@RZ9082 RZ9082 commented Jan 16, 2026

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

@RZ9082 RZ9082 changed the title Pyega3 - Move credentials to requierements Pyega3 - Move credentials to requirements Jan 16, 2026
@RZ9082
Copy link
Collaborator Author

RZ9082 commented Jan 19, 2026

Should I remove the old tests?

bgruening
bgruening reviewed Jan 19, 2026
View reviewed changes
@bgruening bgruening mentioned this pull request Jan 19, 2026
Merged
bgruening
bgruening reviewed Jan 20, 2026
View reviewed changes
@RZ9082 RZ9082 self-assigned this Jan 21, 2026
@RZ9082
Copy link
Collaborator Author

RZ9082 commented Jan 21, 2026

The failing lint is caused by the hidden test parameter.

@RZ9082 RZ9082 moved this from In progress to Ready in Galaxy Tool Developers SIG Board Jan 21, 2026
@RZ9082 RZ9082 marked this pull request as ready for review January 21, 2026 12:18
bgruening
bgruening reviewed Jan 21, 2026
View reviewed changes
@RZ9082 RZ9082 marked this pull request as draft January 22, 2026 16:55
@RZ9082
Copy link
Collaborator Author

RZ9082 commented Jan 23, 2026

@bernt-matthias As credentials a not available as cheetah variables, does this approach look ok for you?

bernt-matthias
bernt-matthias reviewed Jan 23, 2026
View reviewed changes
@RZ9082 RZ9082 marked this pull request as ready for review January 23, 2026 14:14
@RZ9082 RZ9082 mentioned this pull request Jan 26, 2026
Open
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bgruening bgruening bgruening approved these changes

@bernt-matthias bernt-matthias bernt-matthias approved these changes

Assignees

@RZ9082 RZ9082

Labels

None yet

Projects

Galaxy Tool Developers SIG Board
Status: Ready

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@RZ9082 @bgruening @bernt-matthias