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.

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

Hotfix: fixes bug where Roundtrip is not distributed #91

Merged
merged 1 commit into from
Jun 15, 2023
Merged

Hotfix: fixes bug where Roundtrip is not distributed #91

merged 1 commit into from
Jun 15, 2023

Conversation

ilumsden
Copy link
Collaborator

@ilumsden ilumsden commented Jun 2, 2023

Thanks to @vanessalama09, we discovered a bug that causes Hatchet's built-in Roundtrip to not be installed. This hotfix adds the hatchet/external/roundtrip directory to MANIFEST.in to fix this.

@ilumsden ilumsden requested a review from slabasan June 2, 2023 18:57
@ilumsden ilumsden self-assigned this Jun 2, 2023
@ilumsden ilumsden added area-deployment Issues and PRs involving Hatchet's packaging and deployment priority-urgent Urgent priority issues and PRs type-bug Identifies bugs in issues and identifies bug fixes in PRs labels Jun 2, 2023
@slabasan slabasan merged commit 9f02a89 into LLNL:develop Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slabasan slabasan Awaiting requested review from slabasan

Assignees

@ilumsden ilumsden

Labels
area-deployment Issues and PRs involving Hatchet's packaging and deployment priority-urgent Urgent priority issues and PRs type-bug Identifies bugs in issues and identifies bug fixes in PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ilumsden @slabasan