Skip to content
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

Fixed incorrect email on my past comment. #44

Closed
wants to merge 258 commits into from

Conversation

LostbBlizzard
Copy link
Contributor

Hello I'm back again. I remembered that I used my personal email and not the private email GitHub give me and I don't want my personal email to be public for security reasons. This pull request fixes that.

Through I used git force push so I'm not sure if that cause problems with the merge and this also changes all the git hashes.

This is the shell script I used to fix the email in case you needed to know what I ran.

#!/bin/bash

git filter-branch --env-filter '

OLD_EMAIL="[email protected]"
CORRECT_NAME="LostbBlizzard"
CORRECT_EMAIL="[email protected]"

if [ "$GIT_COMMITTER_EMAIL" = "$OLD_EMAIL" ]
then
    export GIT_COMMITTER_NAME="$CORRECT_NAME"
    export GIT_COMMITTER_EMAIL="$CORRECT_EMAIL"
fi
if [ "$GIT_AUTHOR_EMAIL" = "$OLD_EMAIL" ]
then
    export GIT_AUTHOR_NAME="$CORRECT_NAME"
    export GIT_AUTHOR_EMAIL="$CORRECT_EMAIL"
fi
'

Foo and others added 24 commits May 29, 2023 23:32
std::barrier instead omp barrier
samples as custom cmake tarfet triggering the script with proper options

tested for tcp

tested for udp

samples documentation updated and improved
…ythonPopen

Monitor.py script for samples
…CmakeSupport

avoid to fetch samples in readme cmake support
auto detect tests labels in runTests CI action
…d-PR

fixed errors when using wasm emscripten.
avoid custom target command in VisualStudio in favour of a Popen C++ …
tests updated

documentation updated and improved
This was referenced Jul 27, 2024
@andreacasalino
Copy link
Owner

I have regenerated master using the commit with the amended email from your fork.
Current master should be fixed, please check

Andrea

@LostbBlizzard
Copy link
Contributor Author

Just did a git clone and looks like email was fixed. Thanks for fixing it. I'll close this pull request now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants