Skip to content

Update vendor schemas weekly instead of daily#412

Merged
sirosen merged 1 commit intopython-jsonschema:mainfrom
hugovk:weekly-cron
Apr 2, 2024
Merged

Update vendor schemas weekly instead of daily#412
sirosen merged 1 commit intopython-jsonschema:mainfrom
hugovk:weekly-cron

Conversation

@hugovk
Copy link
Copy Markdown
Contributor

@hugovk hugovk commented Apr 2, 2024

re: #409 (comment)

Daily is too noisy, dial down to weekly instead.

I arbitrarily went with Sunday, kept it at 4am UTC:

@hugovk hugovk mentioned this pull request Apr 2, 2024
@@ -1,8 +1,8 @@
name: daily
Copy link
Copy Markdown
Contributor

@edgarrmondragon edgarrmondragon Apr 2, 2024

Choose a reason for hiding this comment

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

While you're at it, you could probably change the title of the PR this creates to something more descriptive than the default Changes by create-pull-request action.

https://github.com/peter-evans/create-pull-request?tab=readme-ov-file#action-inputs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd definitely accept a change for that, but I don't want to block on it. I'm going ahead and merging.

@sirosen
Copy link
Copy Markdown
Member

sirosen commented Apr 2, 2024

Thanks so much for putting this in!

@sirosen sirosen merged commit 3d54204 into python-jsonschema:main Apr 2, 2024
@hugovk hugovk deleted the weekly-cron branch April 2, 2024 14:28
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.

3 participants