Skip to content

Skip unnecessary work when just backing up gists#526

Merged
josegonzalez merged 3 commits into
josegonzalez:masterfrom
mrexodia:incremental-gists
Jul 18, 2026
Merged

Skip unnecessary work when just backing up gists#526
josegonzalez merged 3 commits into
josegonzalez:masterfrom
mrexodia:incremental-gists

Conversation

@mrexodia

Copy link
Copy Markdown
Contributor

No description provided.

@josegonzalez

Copy link
Copy Markdown
Owner

Can you add tests for this new behavior?

@mrexodia
mrexodia force-pushed the incremental-gists branch from 230c1a1 to c49ffef Compare July 18, 2026 10:40
@mrexodia

Copy link
Copy Markdown
Contributor Author

Not exactly sure how the tests work in this repo FYI, but my clanker tried really hard 🥺 I mostly tested by backing up my personal gists(+starred) and it goes much faster now than before.

@josegonzalez

Copy link
Copy Markdown
Owner

I usually defer to @Iamrodos for approval. Need to figure out what the level of perms is to get them as a maintainer without leaking my org secrets or something :D

@josegonzalez

Copy link
Copy Markdown
Owner

Maybe just moving this and python-gitlab-backup to a new org :D

@Iamrodos

Copy link
Copy Markdown
Contributor

Tests are ok.

@Iamrodos

Copy link
Copy Markdown
Contributor

I usually defer to @Iamrodos for approval. Need to figure out what the level of perms is to get them as a maintainer without leaking my org secrets or something :D

Unless you are wanting to avoid the merge button I am happy to just add commentary to PRs. Keeps it simple. Or you could try Triage if you want to take a half step to Maintainer. That can open up some admin without secrets risk.

@josegonzalez
josegonzalez merged commit 822071c into josegonzalez:master Jul 18, 2026
10 checks passed
@mrexodia
mrexodia deleted the incremental-gists branch July 18, 2026 21:16
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