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

Add two ctests to RDASApp #3

Merged
merged 4 commits into from
Nov 2, 2023
Merged

Add two ctests to RDASApp #3

merged 4 commits into from
Nov 2, 2023

Conversation

CoryMartin-NOAA
Copy link
Contributor

This PR:

  • adds two ctests to RDASApp: 1) pycodestyle checks and 2) generate a sample YAML from templates using wxflow
  • automates GitHub Actions to run the ctests with each pull request

@CoryMartin-NOAA CoryMartin-NOAA self-assigned this Nov 1, 2023
@CoryMartin-NOAA CoryMartin-NOAA merged commit 2bf1130 into develop Nov 2, 2023
3 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/yaml-test branch November 2, 2023 13:13
delippi pushed a commit to delippi/RDASApp that referenced this pull request May 23, 2024
@CoryMartin-NOAA
Add two ctests to RDASApp (NOAA-EMC#3)
dca5b47 
* working sample ctests

* fix issue

* change name of action since no cpplint (yet)
@xyzemc xyzemc mentioned this pull request May 28, 2024
Merged
@xyzemc xyzemc mentioned this pull request Jun 10, 2024
Merged
@rrfsbot rrfsbot mentioned this pull request Sep 23, 2024
Closed
This was referenced Oct 17, 2024
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TingLei-NOAA TingLei-NOAA TingLei-NOAA approved these changes

@ShunLiu-NOAA ShunLiu-NOAA ShunLiu-NOAA approved these changes

@hu5970 hu5970 Awaiting requested review from hu5970

@JacobCarley-NOAA JacobCarley-NOAA Awaiting requested review from JacobCarley-NOAA

@delippi delippi Awaiting requested review from delippi

@xyzemc xyzemc Awaiting requested review from xyzemc

@HuiLiu-NOAA HuiLiu-NOAA Awaiting requested review from HuiLiu-NOAA

Assignees

@CoryMartin-NOAA CoryMartin-NOAA

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@CoryMartin-NOAA @ShunLiu-NOAA @TingLei-NOAA