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

api interop layer deploy #1825

Open
wants to merge 7 commits into
base: api-interop-layer
Choose a base branch
from

Conversation

jamestranovich-noaa
Copy link
Collaborator

What does this PR do? 🛠️

Addresses #1791 -- except, in james, and not sandbox, since I don't think we're quite set up on that front (yet)

What does the reviewer need to know? 🤔

This only works in james currently. Visit https://api-weathergov-james.app.cloud.gov/ (by default the app will return { ok: true } for /)

Copy link
Collaborator

@eric-gade eric-gade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

One general question though -- why is the main app only set to 256M? Seems small?

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.

2 participants