Skip to content

docs update#82

Merged
guptaankit015 merged 3 commits into
mainfrom
chore/update-readme-snapshot-docs
May 5, 2026
Merged

docs update#82
guptaankit015 merged 3 commits into
mainfrom
chore/update-readme-snapshot-docs

Conversation

@guptaankit015
Copy link
Copy Markdown
Contributor

@guptaankit015 guptaankit015 commented May 5, 2026

Note

Low Risk
Documentation-only changes that clarify supported runner types and recommended snapshot workflows; no code or behavior changes.

Overview
Updates README.md to clarify that snapshot runners only work on WarpBuild-managed Ubuntu runners (and that snapshot labels are ignored on unsupported runners), document snapshot retention (15 days) and /tmp non-persistence, and add notes about slower boot times.

Replaces prior conditional/container-focused examples with two clearer workflows: clean snapshot creation on main using snapshot.enabled=true plus a gated snapshot-save, and incremental snapshots on feature branches using snapshot.key, along with expanded cleanup guidance (rm -rf creds and git clean -ffdx).

Reviewed by Cursor Bugbot for commit 84b4764. Configure here.

Comment thread README.md
Comment on lines +140 to +151
**Remove untracked files and directories:**

It might be useful to remove some secret files that were added during the job,
before making a snapshot.

- _git clean_: removes untracked files from the local git repo.
- _-f (force)_: forces the removal of files and directories.
- _-f (force again)_: if `git config clean.requireForce true` is present, some files
may not be removed without this flag.
- _-d (directories)_: removes directories not just files.
- _-x (ignore .gitignore)_: removes files and directories that are ignored by git.

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.

do we need to write explaination for the git command here? It looks out of scope for this snapshot doc.

@guptaankit015 guptaankit015 merged commit 4080aad into main May 5, 2026
14 of 19 checks passed
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