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!

@Jelle-S Jelle-S force-pushed the feature/issue-#12 branch from 4b873a7 to dd63bc4 Compare October 2, 2018 10:59
@Jelle-S Jelle-S force-pushed the feature/issue-#12 branch from dd63bc4 to 6ad3dd0 Compare October 2, 2018 11:01
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

@Jelle-S Jelle-S force-pushed the develop branch 9 times, most recently from 1fb33de to c941699 Compare August 6, 2019 11:02
@Jelle-S Jelle-S force-pushed the feature/issue-#12 branch from 9e246c0 to 71541a1 Compare April 23, 2024 08:13
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