Skip to content

Conversation

t2y
Copy link
Contributor

@t2y t2y commented Nov 7, 2023

This might break compatibility, but I guess that most developers want this behavior. What do you think?

@t2y
Copy link
Contributor Author

t2y commented Nov 7, 2023

CI job failed, but it is another issue instead of this PR.

https://github.com/go-ldap/ldap/actions/runs/6781841980/job/18432888679#step:5:67

@t2y
Copy link
Contributor Author

t2y commented Nov 15, 2023

--- FAIL: FuzzParseDN (0.00s)
--- FAIL: FuzzDecodeEscapedSymbols (0.00s)
--- FAIL: FuzzEscapeDN (0.00s)
--- FAIL: FuzzParseDN (0.00s)
--- FAIL: FuzzDecodeEscapedSymbols (0.00s)
--- FAIL: FuzzEscapeDN (0.00s)
--- FAIL: FuzzParseDN (0.00s)
--- FAIL: FuzzDecodeEscapedSymbols (0.00s)
--- FAIL: FuzzEscapeDN (0.00s)

https://github.com/go-ldap/ldap/actions/runs/6781841980/job/18726095280?pr=471

@t2y
Copy link
Contributor Author

t2y commented Nov 16, 2023

Still, three jobs are left. Could you re-run or permit me to run CI jobs?

@t2y
Copy link
Contributor Author

t2y commented Nov 20, 2023

I will rebase from #473 if it would be merged first. Then, CI jobs will be passed.

@t2y t2y force-pushed the refactor-is-error-any-of branch from 3c806a3 to 00afd10 Compare November 21, 2023 00:04
@t2y
Copy link
Contributor Author

t2y commented Nov 21, 2023

As intended.

@johnweldon
Copy link
Member

Thank you again @t2y

@johnweldon johnweldon merged commit 12f15cc into go-ldap:master Nov 21, 2023
@t2y t2y deleted the refactor-is-error-any-of branch November 21, 2023 00:16
gustavoluvizotto pushed a commit to gustavoluvizotto/ldap-fork that referenced this pull request Apr 11, 2024
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