-
Notifications
You must be signed in to change notification settings - Fork 2
Project Synchronization #12
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
base: main
Are you sure you want to change the base?
Conversation
I don't have complete trust in the notion to markdown and markdown to notion python packages, we should probably pin those if rye allows for that and then review before upgrade. I wasn't able to find a well maintained notion<>markdown package that converts both ways and allows to easily separate out the conversion process. |
I haven't gone through everything yet, but one thing I don't really like structurally about this is that it merges the separate scripts together and runs them sequentially. That's:
|
I added a commit that uses GitHub actions matrix strategy to run each sync in parallel that should alleviate above concerns. |
Making a start on getting this off my chest and back into the core repo. This is a pretty major change in terms of code structure and configuring, but shouldn't (tm) break any existing workflows.
It is probably easier to review the code in its source rather than looking at the diff https://github.com/kewisch/thunderbird-notion-scripts/tree/gh_project
I have the GitHub Actions automation running in my repo on the main branch. The cron job is set to every 5 minutes, but in reality runs about every 15 minutes. For my purposes that is (currently) good enough, but might not work for everyone.
I have run the synchronizer with the original services code in my test repo and personal notion instance and it seems to work, but it would probably be a good idea to run it on something that is closer to prod to verify it still works as expected. A notable change is that rather than sleeping between requests I wait until the 429 and then sleep for the indicated number of seconds. I haven't actually triggered rate limiting, so I'm unsure if that works.
I have not tested the bugzilla code at all, are we actively using that? It seemed a bit more experimental.