-
Notifications
You must be signed in to change notification settings - Fork 357
[BitSet] Amend return value of update(with:) method
#538
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
lorentey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! What a thinko
|
Hmph; workflow is not triggering. I'll close and reopen this PR to try to get things in motion -- apologies for the noise. |
|
@aluco100 Please stop reviewing PRs in this repository; it is a nuisance, and I'll be forced to take action if it continues. |
Hey @lorentey the idea to have an open source project is to contribute between each other right? Otherwise this repo should be private if you do not want any external inputs. Please I only want to contribute and if you put a pr ready to review I believe is ready to ship. Saying that I didn't encounter any issues to that so please if you believe I block you please consider this repo as private or please tell me to avoid bothering you. |
|
Posting unsolicited checkmarks to PRs that you are not authoring, for which you are not code owner or a requested reviewer, is going to be regarded as nuisance behavior in essentially any repo. You are causing notifications to be sent to everybody who is involved or even just watching the repo with no content whatsoever. You should not be surprised if you get banned from repos for continuing this behavior, and it is likely your account will eventually be reported to GitHub as well. If you are enthusiastic about a contribution you may consider pushing the button below the PR summary to react with ❤️. You are of course welcome to raise specific concerns--for example, when you notice an error in the code--by writing comments in PRs. For substantive contributions, you may refer to the following link for this repo's specific guidelines: |
This comment was marked as resolved.
This comment was marked as resolved.
66d3bf3 to
e2e2d00
Compare
The implementation and tests didn't match the documentation.
The tests are amended to rely on the expected
Set<Int>results.Checklist