Cosmetic changes : quotes, braces and extracted variable #16

Merged

jlecour merged 2 commits from cosmetic-changes into master

2019-04-25 11:39:41 +02:00

jlecour commented

2019-04-03 21:08:23 +02:00

Owner

Here are the changes that have been removed from a previous PR.

benpro approved these changes 2019-04-04 09:50:28 +02:00

benpro left a comment

Contributor

BTW, it is unnecessary to put braces to every variables if you known that the variable is not concatenated with text.

Like echo "${foo}bar".
echo "$foo"; echo "bar"
There is not need to set braces expansion every time.

But that cause no issue.

BTW, it is unnecessary to put braces to every variables if you known that the variable is not concatenated with text. Like echo "${foo}bar". echo "$foo"; echo "bar" There is not need to set braces expansion every time. But that cause no issue.

benpro commented

2019-04-19 15:28:50 +02:00

Contributor

@vlaborie merge?

Ghost approved these changes 2019-04-23 17:32:19 +02:00

jlecour closed this pull request

2019-04-25 11:39:41 +02:00

jlecour deleted branch cosmetic-changes

2019-04-25 11:40:44 +02:00

No reviewers

No Label

No Milestone

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: evolix/evobackup#16

No description provided.