Skip to content

Use Option.is_some() API#1

Merged
romanb merged 1 commit intowireapp:masterfrom
nremond:patch-1
Apr 18, 2017
Merged

Use Option.is_some() API#1
romanb merged 1 commit intowireapp:masterfrom
nremond:patch-1

Conversation

@nremond
Copy link
Contributor

@nremond nremond commented Apr 7, 2017

No description provided.

@CLAassistant
Copy link

CLAassistant commented Apr 7, 2017

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

Choose a reason for hiding this comment

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

Surely you meant to write return instead of match here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops :-/
I updated my commit

Copy link
Contributor

@twittner twittner left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thank you!

@romanb romanb merged commit e97f7c8 into wireapp:master Apr 18, 2017
smatting pushed a commit that referenced this pull request Apr 20, 2022
* Add nix support.

* Add okta, centrify, generic sso setup howtos.
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.

4 participants