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

comma separated values and documentation #818

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

apoorvachikara
Copy link

No description provided.

@mastermunj
Copy link
Owner

@apoorvachikara Please, do not include package-lock.json file in your commits. That way it won't come in the PR as well.

@apoorvachikara
Copy link
Author

I have removed the package-lock.json. If you check the "file changed" tab - the lock file is removed from there. I removed it long back from the code. Why it is conflicting that the file is not present in the PR?

@mastermunj
Copy link
Owner

Okay, so basically you do not need to delete the file. Let the file be there as it is. Make changes in the other files and commit those files in a branch and raise a PR.

@apoorvachikara
Copy link
Author

I made the changes can you check now? Is it fine 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