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

CONTRIBUTING: update 'Building and testing' instructions #726

Merged
merged 1 commit into from Jan 28, 2020

Conversation

@phil-blain
Copy link
Contributor

@phil-blain phil-blain commented Jan 28, 2020

Overview

TL;DR

The instructions for building the site locally were outdated. I updated them.

Questions

I saw the README talks about script/package that doesn't exist anymore, but I didn't remove it as I couldn't figure the replacement procedure. I can update this branch with changes to the README if that's desired.

Next Steps

  • Update README

Review

@phil-blain phil-blain requested a review from crichID Jan 28, 2020
@brianamarie
Copy link
Contributor

@brianamarie brianamarie commented Jan 28, 2020

Hi @phil-blain! Thank you for opening this pull request. You are right that script/package is no longer there - I believe that is actually a mistake. I'm going to open an issue with that information, and where it can be found, so that it doesn't get lost.

Thank you for this contribution!

@brianamarie brianamarie merged commit 850ab7a into github:master Jan 28, 2020
@NishkarshRaj NishkarshRaj mentioned this pull request Jan 28, 2020
0 of 7 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.