Skip to content
This repository has been archived by the owner on Dec 22, 2018. It is now read-only.

Ability to use fs.chmodSync on the recorded file for #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbergz
Copy link

@mbergz mbergz commented Apr 16, 2018

more flexibility with access permission rights

E.g. useful in CI environments where an automated user is running the tests and saving recordings for other people to access.

readme.md Outdated
@@ -59,6 +59,8 @@ For example, you can do:
* `createSubtitles` (bool): If `true` and singleVideo is also true, will create a SRT subtitles file with the name details of the currently running spec. Defaults to `true`.
The file will be saves to `baseDirectory/protractor-specs.srt`.

* `chmod` (string): Will attempt so set the given chmod on saved video. Defaults to `none` which will use the default permission settings for the file.
Copy link

@yeikel yeikel Apr 16, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following the same style as the other entries in the readme, you should add an example in the readme as well.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

more flexibility with access permission rights
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants