Skip to content

Spelling errors correction #16

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Sahil171102
Copy link

No description provided.

Copy link
Member

@Aman-Codes Aman-Codes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Sahil171102 thank you for your contribution. Could u please revert the unnecessary changes in package.json and package-lock.json. Also please sign your commits to pass the failing test.

This reverts commit 82dda9c.

Signed-off-by: Sahil Gupta <[email protected]>
Signed-off-by: Sahil Gupta <[email protected]>
@Sahil171102 Sahil171102 closed this Dec 5, 2022
@Sahil171102 Sahil171102 reopened this Dec 5, 2022
@Sahil171102
Copy link
Author

Reverted the unnecessary changes and signed the commits

@Aman-Codes
Copy link
Member

Hi @Sahil171102 Could you please run the command
git rebase -i HEAD~6 it will merge all the 6 commits into a single commit. Then running the command git push -f should hopefully fix the failing test cases.

@Sahil171102
Copy link
Author

I merged the commits , Is everything fine now?

@Aman-Codes
Copy link
Member

I merged the commits , Is everything fine now?

But in GitHub it still shows 5 commits :(
image

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.

2 participants