-
-
Notifications
You must be signed in to change notification settings - Fork 235
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
to-hex payloadprocessor #178
Conversation
This pull request introduces 2 alerts when merging 367410f into f711c0d - view on LGTM.com new alerts:
|
Thanks for sharing this :) |
This pull request introduces 1 alert when merging 1bf5fe4 into f711c0d - view on LGTM.com new alerts:
|
thanks, this is my first ever pull :) |
I think I'm done with edits? |
Looks good to me :) |
fixed typo, function name and added to changelog |
is there anything else required? |
Could you squash the commits? (The first is using your email, fine if you want it public though.) |
how do I do that? |
ok thanks, I'm using Bash rather than Eclipse. |
The middle link has CLI details. These might also help you: |
If you're having trouble, let me know I can tackle it. |
thanks, for the help. I used: but i'm getting this issue when i try to push:
|
You need to force push ( |
Keep getting: tried: |
Check |
git status
after several attempts and a few rm ./community-scripts -fr and clone again i think i almost got it. |
36df5b6
to
f6cf23d
Compare
ok i think i've squashed the commits, and removed email. |
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.
Thanks!
The committer name/email is probably not correct. |
thanks, I used this to fix for reference. https://stackoverflow.com/questions/3042437/how-to-change-the-commit-author-for-one-specific-commit |
Thank you! |
Useful script I used for Natas 19 Challenge:
https://youtu.be/aZ3ZfAhdqT8