Skip to content

precommit update and cron job#87

Merged
dgarciabriseno merged 2 commits into
mainfrom
CI
Dec 9, 2022
Merged

precommit update and cron job#87
dgarciabriseno merged 2 commits into
mainfrom
CI

Conversation

@nabobalis

@nabobalis nabobalis commented Dec 8, 2022

Copy link
Copy Markdown
Member

Pin tox<4.0 just to check as well

@nabobalis
nabobalis marked this pull request as ready for review December 8, 2022 19:23
@nabobalis

Copy link
Copy Markdown
Member Author

Tests run, failure is a missing source number.

@dgarciabriseno

Copy link
Copy Markdown
Collaborator

Yep, failure was a change on beta. Had to update the SUVI IDs there for my other PR.
I made #88 just as a sanity check to make sure everything is good after this change and my other change. Once that passes I'll merge in these two.

Thanks for figuring this out! I didn't even think about the tox version, I was getting the same failure locally and it wasn't making any sense to me since there wasn't any real error message, it just said failed!

- cron: '0 7 * * *' # Every day at 07:00 UTC

jobs:
dispatch_workflows:

@dgarciabriseno dgarciabriseno Dec 9, 2022

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is neat, didn't know you could schedule workflows like this. Good idea.

@dgarciabriseno
dgarciabriseno merged commit 6bfe365 into main Dec 9, 2022
@dgarciabriseno
dgarciabriseno deleted the CI branch December 9, 2022 15:53
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