Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 12:24
    scnale edited #13928
  • 12:05
    github-actions[bot] closed #9987
  • 12:05
    github-actions[bot] labeled #9987
  • 12:05
    github-actions[bot] commented #9987
  • 12:05
    github-actions[bot] closed #9907
  • 12:05
    github-actions[bot] labeled #9907
  • 12:05
    github-actions[bot] commented #9907
  • 12:05
    github-actions[bot] closed #9677
  • 12:05
    github-actions[bot] labeled #9677
  • 12:05
    github-actions[bot] commented #9677
  • 12:04
    github-actions[bot] closed #9673
  • 12:04
    github-actions[bot] labeled #9673
  • 12:04
    github-actions[bot] commented #9673
  • 12:04
    github-actions[bot] closed #9626
  • 12:04
    github-actions[bot] labeled #9626
  • 12:04
    github-actions[bot] commented #9626
  • 12:04
    github-actions[bot] closed #9621
  • 12:04
    github-actions[bot] labeled #9621
  • 12:04
    github-actions[bot] commented #9621
  • 12:04
    github-actions[bot] closed #9462
Tushar Gupta
@tushar-corbic:matrix.org
[m]
@cameel:
cameel
@cameel:matrix.org
[m]
I can find you something easy to start.
Tushar Gupta
@tushar-corbic:matrix.org
[m]
That would be great
cameel
@cameel:matrix.org
[m]
How strong is your C++?
Tushar Gupta
@tushar-corbic:matrix.org
[m]
I would say something above beginner but definately less than intermediate
currently reading The C++ standard library book by Nicolai M. Josuttis
So I think this project would be great opportunity to apply all these concepts in real world
cameel
@cameel:matrix.org
[m]
ok, so how about this one? ethereum/solidity#10210
This should be relatively easy, I'll post a comment in the issue about how to do it.
Tushar Gupta
@tushar-corbic:matrix.org
[m]
okay sure, I will definately look into ti
1 reply
it*
Tushar Gupta
@tushar-corbic:matrix.org
[m]
Thank you, I really appreciate your efforts
cameel
@cameel:matrix.org
[m]
Also see https://docs.soliditylang.org/en/latest/contributing.html#contributing for how to set up your environment.
You can ask on this channel if you run into any problems with that.
By the way, what platform are you on?
Linux, Mac, Windows?
Tushar Gupta
@tushar-corbic:matrix.org
[m]
Linux
cameel
@cameel:matrix.org
[m]
ok, great
That's the one you should have least problems with.
Tushar Gupta
@tushar-corbic:matrix.org
[m]
Yess
Nikola Matić
@nikola.matic:matrix.org
[m]
r0qs, you gonna merge this? ethereum/solidity#13910
Or does it need more work?
r0qs
@r0qs:matrix.org
[m]
We can merge it. But I was waiting to see if someone else also supports your suggestion to only run the command line tests on clang, which would make such fix unnecessary.
1 reply
Nikola Matić
@nikola.matic:matrix.org
[m]
Sure thing, no rush.
Was just wondering if you forgot about it :)
cameel
@cameel:matrix.org
[m]
Though I don't know the full context. I'm going to look at that PR a bit later but it didn't seem to have anything bad in it last time I looked so it does not necessarily have to wait form .
If Nikola Matić thinks it's fine, I'm ok with merging it now.
cameel
@cameel:matrix.org
[m]

matheus.pit: Can you take ethereum/solidity#13908?

I think we should merge ethereum/solidity#13855 and it will be best if we can do both fixes in the same release.

We still have some decisions to make regarding #13908 (i.e whether to disallow using {L.f} for private or whether to make it work like internal) so keep that in mind but I think it should go in soon whichever way we choose.

1 reply
StackenBotten
@stackenbotten
❌ [nightly] Job t_ubu_asan_cli failed on develop. Please see build 1259180 for details.
3 replies
r0qs
@r0qs:matrix.org
[m]
Yes, for that one we need to wait. But I'm pretty sure it will work
StackenBotten
@stackenbotten
❌ [main] Job t_native_test_ext_gnosis (run 4/6) failed on develop. Please see build 1259218 for details.
❌ [main] Job t_native_test_ext_gnosis (run 1/6) failed on develop. Please see build 1259218 for details.
❌ [main] Job t_native_test_ext_gnosis (run 3/6) failed on develop. Please see build 1259218 for details.
❌ [main] Job t_native_test_ext_gnosis (run 2/6) failed on develop. Please see build 1259218 for details.
Nikola Matić
@nikola.matic:matrix.org
[m]
Reeeee
1 reply
r0qs
@r0qs:matrix.org
[m]
haha
Nikola Matić
@nikola.matic:matrix.org
[m]
By the way, the gnosis failure is (almost definitely) due to the PR that was merged yesterday (in their repository) - safe-global/safe-contracts#502, however, I can't replicate it locally (again in their repository). Another thing with gnosis is that we still use npm to build and run, whereas the project itself uses yarn. I have (and will be) trying to reproduce this so I can open an issue downstream.
1 reply
In the meantime, a temporary fix could be to checkout a commit instead of the branch, and this allows the external test to pass, however, I'm not particularly keen on this approach as we'll essentially be using a frozen snapshot of the project until the issue is properly dealt with.
cameel
@cameel:matrix.org
[m]
Yeah, I'd rather fix these errors as they come. It's annoying but I think that fixing multiple errors in multiple tests at once when you try to update is an even bigger headache.
There should not be that much difference between npm and yarn though.
Probably good to use whatever the project uses in their README but these projects switch between them all the time.
cameel
@cameel:matrix.org
[m]
Release checklist update: ethereum/solidity#13926
StackenBotten
@stackenbotten
❌ [main] Job t_native_test_ext_gnosis (run 3/6) failed on develop. Please see build 1259577 for details.
❌ [main] Job t_native_test_ext_gnosis (run 4/6) failed on develop. Please see build 1259577 for details.
❌ [main] Job t_native_test_ext_gnosis (run 2/6) failed on develop. Please see build 1259577 for details.
❌ [main] Job t_native_test_ext_gnosis (run 1/6) failed on develop. Please see build 1259577 for details.
StackenBotten
@stackenbotten
❌ [main] Job t_native_test_ext_gnosis (run 3/6) failed on develop. Please see build 1259643 for details.
❌ [main] Job t_native_test_ext_gnosis (run 4/6) failed on develop. Please see build 1259643 for details.
❌ [main] Job t_native_test_ext_gnosis (run 1/6) failed on develop. Please see build 1259643 for details.
❌ [main] Job t_native_test_ext_gnosis (run 2/6) failed on develop. Please see build 1259643 for details.