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

2091 + Github Action to test #2093

Merged
merged 15 commits into from
Aug 25, 2021
Merged

2091 + Github Action to test #2093

merged 15 commits into from
Aug 25, 2021

Conversation

WardF
Copy link
Member

@WardF WardF commented Aug 24, 2021

This PR is based on #2091 and includes additional github actions to test the changes made therein.

DennisHeimbigner and others added 3 commits August 24, 2021 12:27
re: Github Issue: #2085

The file ncdap_test/manyurls.h was not added to the repository.
It is controlled by the --enable-dap-long-tests option.
Solution: add the file.
@WardF WardF added this to the 4.8.2 milestone Aug 24, 2021
@WardF WardF self-assigned this Aug 24, 2021
@WardF
Copy link
Member Author

WardF commented Aug 25, 2021

The issue appears to be one of a race condition between a couple of tests. Sorting it out now.

@WardF WardF merged commit 6ea721d into master Aug 25, 2021
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