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

Update integration test file #422

Merged
merged 1 commit into from
Nov 29, 2023
Merged

Update integration test file #422

merged 1 commit into from
Nov 29, 2023

Conversation

haoming29
Copy link
Contributor

Fixes #420

@haoming29 haoming29 added this to the v7.3.0 milestone Nov 29, 2023
@haoming29 haoming29 added the test Improvements to the test suite label Nov 29, 2023
turetske
turetske approved these changes Nov 29, 2023
View reviewed changes
Copy link
Collaborator

@turetske turetske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on paper. If the test passes then I would say it's good to merge.

@turetske turetske merged commit 2cd22a4 into PelicanPlatform:main Nov 29, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@turetske turetske turetske approved these changes

Assignees
No one assigned
Labels
test Improvements to the test suite
Projects
None yet
Milestone
v7.3.0
Development

Successfully merging this pull request may close these issues.

Update integration test with new test file
2 participants
@haoming29 @turetske