Skip to content
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

Jump to bottom

Create zfssnap.md #3

Merged
merged 2 commits into from Jul 18, 2019
Merged

Create zfssnap.md #3

merged 2 commits into from Jul 18, 2019

Conversation

Reviewers

@beanpole135 beanpole135

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@jdrch @beanpole135
@jdrch
Copy link
Contributor

@jdrch jdrch commented Jul 16, 2019

Simple, GUI-based guide to setting up ZFS snapshots on Project Trident with zfsnap.
@jdrch
Create zfssnap.md
1843038 
How to set up ZFS snapshots on Project Trident with zfsnap
beanpole135
beanpole135 requested changes Jul 17, 2019
View changes
Copy link
Member

@beanpole135 beanpole135 left a comment

Looks pretty good!
The only thing I would recommend you change/add is to put a little author/date section near the top of the article. We like to make sure the right people get credit for their work... ;-)
Once you add that ownership blurb, I will go ahead and merge this for you.

@jdrch
Update zfssnap.md
d1e32b0 
Changed wording and syntax a bit to be more "just the facts" and proper.
@jdrch
Copy link
Contributor Author

@jdrch jdrch commented Jul 18, 2019

Looks pretty good!
The only thing I would recommend you change/add is to put a little author/date section near the top of the article. We like to make sure the right people get credit for their work... ;-)
Once you add that ownership blurb, I will go ahead and merge this for you.

Thanks! Done, I think :)

@beanpole135 beanpole135 merged commit 76ae34d into project-trident:master Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment