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

Removing .NET 7 from tests, removing outdated tests and increasing timeout-minutes #350

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

nicrowe00
Copy link
Contributor

@nicrowe00 nicrowe00 commented Jun 4, 2024

Cleaning up tests by removing references to .NET 7 and removing tests
made for older, now unsupported versions of .NET. Also increasing
the timeout limit for install .NET and test dependencies due to frequent
timeouts on Fedora.

@nicrowe00 nicrowe00 requested a review from tmds June 4, 2024 08:37
@nicrowe00 nicrowe00 changed the title Removing .NET 7 from tests and removing outdated tests. Removing .NET 7 from tests, removing outdated tests and increasing timeout-minutes Jun 4, 2024
@nicrowe00 nicrowe00 requested a review from omajid June 4, 2024 13:05
@omajid
Copy link
Member

omajid commented Jun 5, 2024

I opened #351 to remove the failing centos stream 8 legs. CentOS Stream 8 is EOL: https://blog.centos.org/2023/04/end-dates-are-coming-for-centos-stream-8-and-centos-linux-7/

timeout-minutes

Cleaning up tests by removing references to .NET 7 and removing tests
made for older, now unsupported versions of .NET. Also increasing
the timeout limit for install .NET and test dependencies due to frequent
timeouts on Fedora.
Copy link
Member

@omajid omajid left a comment

Choose a reason for hiding this comment

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

Thank you!!

@nicrowe00 nicrowe00 merged commit 6bc109b into redhat-developer:main Jun 7, 2024
15 of 16 checks passed
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.

2 participants