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

Create ci.yml #61

Merged
merged 11 commits into from
Oct 15, 2021
Merged

Create ci.yml #61

merged 11 commits into from
Oct 15, 2021

Conversation

yardasol
Copy link
Contributor

This PR adds a file, ci.yml for continuous integration using GitHub actions.

Closes #60.

@yardasol
Copy link
Contributor Author

The CI shows that the 3 tests in app.py are failing. These are the same errors as I got here on my local machine. Can anyone reproduce? @LukeSeifert how do the tests on your build perform?

@yardasol
Copy link
Contributor Author

I'm going to merge so I can see if my fixes in #63 do indeed pass.

@yardasol yardasol merged commit faf00f3 into master Oct 15, 2021
@yardasol yardasol deleted the ci-init branch March 9, 2022 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@munkm munkm Awaiting requested review from munkm

@gwenchee gwenchee Awaiting requested review from gwenchee

@smpark7 smpark7 Awaiting requested review from smpark7

@LukeSeifert LukeSeifert Awaiting requested review from LukeSeifert

Assignees
No one assigned
Labels
Comp:Analysis Difficulty:2-Challenging Priority:2-Normal Type:Test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move continuous integrations testing to Github Actions under arfc/saltproc
1 participant
@yardasol