Skip to content

Fix use of package_name attribute on CondaTarget objects#1424

Merged
mvdbeek merged 1 commit intogalaxyproject:masterfrom
nsoranzo:fix_CondaTarget_package_name
Jan 30, 2024
Merged

Fix use of package_name attribute on CondaTarget objects#1424
mvdbeek merged 1 commit intogalaxyproject:masterfrom
nsoranzo:fix_CondaTarget_package_name

Conversation

@nsoranzo
Copy link
Member

In Galaxy 23.1 galaxy.tool_util.deps.mulled.util.build_target() was changed to return a CondaTarget object instead of a (removed) Target.

To avoid complications, this requires a bump to the galaxy-util and galaxy-tool-util packages to >=23.1 .

Also, add type annotations to most related code.

Fix galaxyproject/galaxy#17378 .

In Galaxy 23.1 `galaxy.tool_util.deps.mulled.util.build_target()`
was changed to return a `CondaTarget` object instead of a (removed)
`Target`.

To avoid complications, this requires a bump to the galaxy-util and
galaxy-tool-util packages to >=23.1 .

Also, add type annotations to most related code.

Fix galaxyproject/galaxy#17378 .
@mvdbeek mvdbeek merged commit 50a6485 into galaxyproject:master Jan 30, 2024
@mvdbeek
Copy link
Member

mvdbeek commented Jan 30, 2024

Thank you!

@nsoranzo nsoranzo deleted the fix_CondaTarget_package_name branch January 30, 2024 14:47
@bernt-matthias
Copy link
Collaborator

Thanks a lot @nsoranzo.

Should we unpin the planemo monitor again?

@nsoranzo
Copy link
Member Author

Should we unpin the planemo monitor again?

Yes, I think so.

bernt-matthias added a commit to bernt-matthias/planemo-monitor that referenced this pull request Jan 30, 2024
@bernt-matthias
Copy link
Collaborator

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.

CondaTarget misses package_name property

3 participants