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

Infrastructure bug fixes #491

Merged
merged 6 commits into from
Jul 20, 2020
Merged

Infrastructure bug fixes #491

merged 6 commits into from
Jul 20, 2020

Conversation

rafmudaf
Copy link
Collaborator

Feature or improvement description
This pull request fixes a bug in the linearization regression test driver script and updates the baselines for two regression test cases which were incorrectly updated in #350.

Related issue, if one exists
No issue but the regression tests currently fail.

Impacted areas of the software
Testing

Test results, if applicable
See GitHub Actions.

@rafmudaf rafmudaf force-pushed the dev branch 3 times, most recently from 5e1fb79 to 356905f Compare July 14, 2020 14:51
@rafmudaf rafmudaf changed the title Dev Infrastructure bug fixes Jul 14, 2020
@rafmudaf rafmudaf marked this pull request as draft July 17, 2020 15:57
@rafmudaf rafmudaf marked this pull request as ready for review July 20, 2020 16:30
rafmudaf added 6 commits July 20, 2020 13:29
Currently, incoming pull requests run a change against NREL's OpenFAST repository.
This works well for pull requests, but prevents the CI from running against code in development in another fork.
They consistently fail on GH Actions but pass everywhere else :(
GitHub now has "draft" pull requests so all new PR's should either
be draft or ready to merge.
@rafmudaf rafmudaf merged commit 67daa66 into OpenFAST:dev Jul 20, 2020
@rafmudaf rafmudaf self-assigned this Oct 7, 2020
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.

1 participant