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.

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

Feat: Added New Tests #870

Closed
wants to merge 2 commits into from
Closed

Conversation

Yola21
Copy link

@Yola21 Yola21 commented Nov 2, 2023

Added tests to Collections Test

@bdemers
Copy link
Member

bdemers commented Nov 2, 2023

It looks like the htmlReport folder should not have been checked in, can you remove it?

@Yola21
Copy link
Author

Yola21 commented Nov 2, 2023

It looks like the htmlReport folder should not have been checked in, can you remove it?

Yeah, sure. I generated the report but forgot to remove it.

@lhazlewood
Copy link
Contributor

@Yola21 thanks for updating the PR! Looks like there's a minor Groovy-on-JDK7 syntax error.

@lhazlewood
Copy link
Contributor

@Yola21 any update?

@Yola21
Copy link
Author

Yola21 commented Nov 19, 2023

@Yola21 any update?

@lhazlewood Yes, I am working on this, was caught up in few other stuff. I will update it soon.

@lhazlewood
Copy link
Contributor

@Yola21 no worries, take your time, I just wanted to ensure you were aware of it 😉

@Yola21 Yola21 closed this Dec 17, 2023
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