-
Notifications
You must be signed in to change notification settings - Fork 14.6k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
docs: Fix typo on line 967 #17785
docs: Fix typo on line 967 #17785
Conversation
THis looks great, it looks like you don't have a package-lock.json. Could you add one? if your master is up to date you could do: |
Wonderful! Let me know when its all green and I will merge :) |
Okay. I used git checkout master superset-frontend/package-lock.json to retrieve the original file, then SKIP=check-added-large-files with this commit to disregard the file size check and push the file. |
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.
Can you add a summary?
* Line 976 was wakeup, now wake up * Removed package lock to test environment * Return original package-lock.json
* Line 976 was wakeup, now wake up * Removed package lock to test environment * Return original package-lock.json
docs: correct spelling of CONTRIBUTING.MD
SUMMARY
Changed "wakeup" to "wake up" on line 967.