Skip to content

update the const LDAPAuthDomain to have type AuthDomain#5

Open
tx0c wants to merge 1 commit intocouchbase:masterfrom
tx0c:patch-1
Open

update the const LDAPAuthDomain to have type AuthDomain#5
tx0c wants to merge 1 commit intocouchbase:masterfrom
tx0c:patch-1

Conversation

@tx0c
Copy link
Contributor

@tx0c tx0c commented Nov 18, 2019

in a const section, each new const does not inherit previous line's type;

https://play.golang.org/p/_vp5ROMdP6f

	fmt.Printf("%[1]T %#[1]v\n", InternalAuthDomain)
	fmt.Printf("%[1]T %#[1]v\n", LDAPAuthDomain)

runs

main.AuthDomain "local"
string "ldap"

https://play.golang.org/p/_vp5ROMdP6f

```go
	fmt.Printf("%[1]T %#[1]v\n", InternalAuthDomain)
	fmt.Printf("%[1]T %#[1]v\n", LDAPAuthDomain)
```

runs

```
main.AuthDomain "local"
string "ldap"
```
ns-codereview pushed a commit that referenced this pull request Nov 19, 2019
https://play.golang.org/p/_vp5ROMdP6f

```go
	fmt.Printf("%[1]T %#[1]v\n", InternalAuthDomain)
	fmt.Printf("%[1]T %#[1]v\n", LDAPAuthDomain)
```

runs

```
main.AuthDomain "local"
string "ldap"
```

Change-Id: Idc693a26cb3a0f31643d3966f5e2bfd7ac282525
Reviewed-on: http://review.couchbase.org/118145
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Tommie McAfee <tommie@couchbase.com>
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.

1 participant