@sgarg_gitlab Thank you!!
Once the MR is merged can I see any deployment or dashboard?
@gitlab-bot ready @sgarg_gitlab
@missy-gitlab Got it thanks
Hey @nick_vh I was not able to contribute in this Hackathon.
@gitlab-bot ready @missy-gitlab
Akash Paudel (8f63d25b) at 09 Mar 09:48
Merge branch gitlab:master into 239356-akashpaudel/fix-Lint/UnusedM...
... and 61 more commits
@justin_ho Fix applied thanks for following
Fix applied
Hey @missy-gitlab I have made changes as per your request. I have amended the same commit. Do we generally recommend amending commits or do we prefer adding new commit after change request?
Akash Paudel (eaf571cb) at 09 Mar 09:45
Fix Lint/UnusedMethodArgument offenses
... and 5463 more commits
@gitlab-bot ready @missy-gitlab I have address your comments
@missy-gitlab same as above response
@missy-gitlab when I look at the method contract def with_captcha_check_common(spammable:, captcha_render_lambda:, &block) it contains &block. Auto correct was removing &block and to prevent it I had to use block to ensure that it does not have Lint/UnusedMethodArgument offenses.
It was also recommended by @gitlab-bot
This is my first MR, I need help understanding. How to tag reviewer? So that MR can be merged.
@gitlab-bot help
Akash Paudel (630902d4) at 10 Feb 05:39
Updated readme with contributor banner
Akash Paudel (a4983f46) at 10 Feb 05:37
Initial commit