obsidian-digital-garden icon indicating copy to clipboard operation
obsidian-digital-garden copied to clipboard

[Feature Request] Combine notes into one commit while publishing multiple notes

Open Mowd opened this issue 2 years ago • 6 comments

Publishing multiple notes creates a commit that only contains one note. If I publish multiple, let's say, 5000+ notes at once, then that will be a long long time to wait. I think combine multiple notes into one commit will significantly improve the speed.

CleanShot_2022-12-02_02 12 56

Mowd avatar Dec 01 '22 18:12 Mowd

Yes, this is super needed. I have a use case where I'm deploying to a Kubernetes cluster via Git Actions rather than Vercel or Netlify, and it is triggered by changes to the main brainch, so yesterday there were like 100 workflows all queued up at the same time. I definitely did not want that

image

ethanimproving avatar Mar 30 '23 14:03 ethanimproving

Moreover, Vercel's free accounts have limited builds per day. Each time the Obsidian repo is re-pulled from the repository, digital-garden will think that every published file needs to be updated. This is very time consuming.

1244919208 avatar May 14 '23 12:05 1244919208

This is still super needed! Would be a game changer.

Esurielt avatar Jan 17 '24 21:01 Esurielt

@oleeskild any plans on this? I can work on implementation (since it's deeply disturbing to have many unnecessary vercel builds), but would you actually merge my PR?

neoromantic avatar Mar 12 '24 08:03 neoromantic

I have recently started using the plugin, and I couldn't agree more. I don't see any benefit to not grouping multiple changes into one commit. To have one per file is counterproductive when Vercel is involved.

jdhorner avatar Mar 26 '24 12:03 jdhorner

I've finally gotten around to implement this. I've released it as a Beta version for now, that can be installed via BRAT. Any feedback, either here or in the Discord is much appreciated 🙏

Release

oleeskild avatar May 07 '24 15:05 oleeskild