Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Localize download delete confirmation #7988

Merged
merged 1 commit into from
Mar 31, 2017
Merged

Localize download delete confirmation #7988

merged 1 commit into from
Mar 31, 2017

Conversation

cndouglas
Copy link

@cndouglas cndouglas commented Mar 30, 2017

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Fixes #7958

@luixxiul
Copy link
Contributor

would you please squash the commits?

@cndouglas
Copy link
Author

Squashed.

By the way, I think we should consider dropping the squash policy since we have squash on merge in GitHub.
image

@luixxiul
Copy link
Contributor

@liunkae I don't like the feature as sometimes commits are squashed by the merger before the author does. Also it breaks by default the verified signature of commits, which I do not want to see.

Copy link
Contributor

@luixxiul luixxiul left a comment

Choose a reason for hiding this comment

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

Tested with another language, looking good.

@luixxiul luixxiul merged commit 7c2e101 into brave:master Mar 31, 2017
@cndouglas
Copy link
Author

@luixxiul Thanks! You also make good points about commit squashing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants