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

Fixes #12: Set useTmpDir to true. #13

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

Jelle-S
Copy link
Collaborator

@Jelle-S Jelle-S commented Sep 27, 2018

No description provided.

@Jelle-S Jelle-S requested a review from daften September 27, 2018 10:06
Since options can't have a default value of `true` (see
https://robo.li/getting-started/#options), we had to revert the option
for the command, so that it still uses the temporary directory to create
the archive.
Copy link
Contributor

@daften daften left a comment

Choose a reason for hiding this comment

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

I'd wait to merge this and discuss this in the technical meeting. Also: CHANGELOG will be needed now definitely!

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Moved 'Fixed' lines to separate sections.
Copy link
Contributor

@daften daften left a comment

Choose a reason for hiding this comment

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

See https://keepachangelog.com/en/1.0.0/ for the keywords to use. @Jelle-S

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.

2 participants