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

Add bin/www express script and updated env variable #205

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

Conversation

fred-cardoso
Copy link
Member

Added the missing bin/www express script that is present on the package.json start script.
If this is used as a sample, running npm start after npm install will fail without the bin/www script.

Updated .gitignore to include the /bin folder

It's not possible to re-include a file if a parent directory of that file is excluded (source) so I opted to remove the bin/ folder from the .gitignore file.
Future /bin outputs can be specifically excluded. None are present at the moment.

Updated the environment variable referencing the HTTP 2.0 Port from portHttp2 to HTTP20_ONLY_PORT, matching the documentation/README.

Updating .gitignore to include the /bin folder
Updated the HTTP 2.0 gRPC port environment variable to match the documentation that points out to HTTP20_ONLY_PORT.
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.

1 participant