sojung.kim

Results 15 comments of sojung.kim

I have a similar question too🥲 `pr_repo = g.get_repo(repo_name)` ``` "/Users/mac/.anyenv/envs/pyenv/versions/3.9.9/lib/python3.9/http/client.py", line 1258, in putheader values[i] = one_value.encode('latin-1') UnicodeEncodeError: 'latin-1' codec can't encode character '\u201c' in position 6: ordinal not...

@Buzzardo Hi, I want to contribute this issue. Can I do it?  😄 😄 😄

Hi, @AlexanderWert This is my first PR, could you possibly review it? Please, I'm asking for your help

@mp911de I kindly request your review of this pull request. Your feedback would be greatly appreciated. 🙇🏻

I tested everything locally and it passed, but it failed on GitHub Actions. I'll make the necessary fixes and commit again soon.

Hey everyone, Creating a new Slots object to handle null slots sounds like a good idea. However, I think using `BitSet(0)` when slots is null can also safely avoid NPEs....

I acknowledge the necessity of creating a separate `Slots` object. However, modifying the constructor of `RedisClusterNode` to include this new `Slots` class would require changes to all existing code that...

I confirmed that my pull request failed during the integration tests. It seems like the failure is related to `HashCommandIntegrationTests`, which doesn’t seem to be directly connected to the changes...

> > There was a change in the redis/redis codebase that broke the tests, it has been since fixed. I've started the pipeline again. > > > Actually - my...