-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fix#16 #31
Fix#16 #31
Conversation
💖 Thanks for opening this pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix.Examples of commit messages with semantic prefixes:-
|
LGTM, can you squash your commits and follow the Semantic Commit messages guidelines too? |
Should I make another PR? Can you please elaborate a lil bit? |
Sure! Essentially you need to use the You can google or ask here if you have any issues! |
Hey, you neednt close this PR. You can make the changes in the same one |
ok i'll do it. thanks for the help |
Can I just close this PR and create a new one with all the required changes? It's a bit confusing for me as I'm new to git. |
Sure whatever suits you
…On Mon, Apr 20, 2020, 23:03 Pranav Shekhar ***@***.***> wrote:
Can I just close this PR and create a new one with all the required
changes? It's a bit confusing for me as I'm new to git.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#31 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFZ6TS2PB75YIXUB3WM2SSTRNSBP5ANCNFSM4MGDQ7LQ>
.
|
Description
Renaming the file config.js according to the standard. Changed the filename to util.js
Fixes #16
Type of Change:
Change in filename.