-
Notifications
You must be signed in to change notification settings - Fork 95
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
change the year from 2019 to 2020. #51
Conversation
Update Hackathon Section
Update Hackathon section
This reverts commit 40ed49e.
Updated Elastic Logo
Squash your commits , you have 7 commits for |
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.
Why adding an image file?
Also the issue no. is linked in PR description not in PR title, |
@shivaylamba @edjustice @HeinKhantZaw please review PR's carefully. |
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.
Changes to be squashed
Elastic.png is already there in the main branch, i m not adding it. I did this pr long ago and after that Elastic.png was added. i merged the new changes made by the new PR(6th commit) and after that it started showing. |
please rebase your branch and then squash the commits. |
cloning the forked repo results in chaging the img file name(lowercase to uppercase). i m unable to rebase as local changes need to be commit first and doing so will leads to another change in image file. |
If you are having trouble in reading just reset your branch. As this is a very minor change . First abort if any rebase in process ,
|
fix issue #50
Before:
After: