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

Update generated render.yaml #6771

Merged
merged 3 commits into from
Nov 4, 2022
Merged

Update generated render.yaml #6771

merged 3 commits into from
Nov 4, 2022

Conversation

dphuang2
Copy link

@dphuang2 dphuang2 commented Nov 1, 2022

I was running into an error that said --production=false is not a flag. After changing TRUE -> true it went away. Not sure what the best way to test this change though.

I was running into an error that said `--production=false` is not a flag. After changing `TRUE` -> `true` it went away. Not sure what the best way to test this change though.
@Tobbe
Copy link
Member

Tobbe commented Nov 1, 2022

That's interesting.
That setting was introduce here https://github.com/redwoodjs/redwood/pull/6375/files

@Tobbe
Copy link
Member

Tobbe commented Nov 1, 2022

Looks like it should be lowercase true
https://community.render.com/t/support-for-yarn-2-workspaces/135/4

@Tobbe
Copy link
Member

Tobbe commented Nov 1, 2022

Not sure what the best way to test this change though.

I'm afraid we don't have a good way to test this 🙁 Manual testing is the way to go for now I think

@Tobbe Tobbe added release:fix This PR is a fix fixture-ok Override the test project fixture check labels Nov 1, 2022
@simoncrypta
Copy link
Collaborator

Hey sorry everyone, I tested the change in the last PR by copy past in my config and I probably didn't do it correctly.
I know we have this repo to test deploys https://github.com/redwoodjs/deploy-target-ci
I think it needs some update to be able to test change in config file.

Copy link
Contributor

@jtoar jtoar left a comment

Choose a reason for hiding this comment

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

@simoncrypta yeah I just pinged David about the config in the deploy target CI repo, will see what he says. In the meantime, I tested this manually and it works.

@jtoar
Copy link
Contributor

jtoar commented Nov 4, 2022

Also, render docs here: https://render.com/docs/environment-variables#node-1

@jtoar jtoar merged commit c072358 into redwoodjs:main Nov 4, 2022
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Nov 4, 2022
github-actions bot pushed a commit that referenced this pull request Nov 4, 2022
I was running into an error that said `--production=false` is not a flag. After changing `TRUE` -> `true` it went away. Not sure what the best way to test this change though.

Co-authored-by: Simon Gagnon <git@simoncrypta.dev>
Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
jtoar added a commit that referenced this pull request Nov 4, 2022
I was running into an error that said `--production=false` is not a flag. After changing `TRUE` -> `true` it went away. Not sure what the best way to test this change though.

Co-authored-by: Simon Gagnon <git@simoncrypta.dev>
Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
dac09 added a commit that referenced this pull request Nov 7, 2022
…aching

* 'main' of github.com:redwoodjs/redwood: (21 commits)
  [Tutorial]: Fix Typescript code blocks inconsistency (#6801)
  chore: update all contributors
  Custom auth: Fix comment in template (#6804)
  fix(deps): update dependency eslint to v8.26.0 (#6785)
  [CRWA]: Switch to using enquirer, add engine compatibility override option (#6723)
  (docs): Minor Command update about Storybook (#6722)
  docs: Add mocking useLocation to docs (#6791)
  Update generated render.yaml (#6771)
  fix flightcontrol config template (#6789)
  fix: publish canary using premajor (#6794)
  Strip resetToken and resetTokenExpiresAt from dbAuth forgotPassword handler (#6778)
  Fix WebAuthn when event body is base64 encoded (like when deploying to Vercel) (#6757)
  fix(deps): update jest monorepo (#6787)
  fix(deps): update dependency react-hook-form to v7.39.1 (#6786)
  fix(deps): update dependency fastify to v4.9.2 (#6781)
  fix(deps): update dependency @apollo/client to v3.7.1 (#6780)
  chore: fix and rebuild test project fixture (#6775)
  fix: add prisma resolutions to tutorial e2e test proj (#6772)
  fix(deps): update prisma monorepo to v4.5.0 (#6485)
  Fix dbauth webauthn template (redundant type import) (#6769)
  ...
jtoar added a commit that referenced this pull request Nov 8, 2022
I was running into an error that said `--production=false` is not a flag. After changing `TRUE` -> `true` it went away. Not sure what the best way to test this change though.

Co-authored-by: Simon Gagnon <git@simoncrypta.dev>
Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
jtoar added a commit that referenced this pull request Nov 8, 2022
I was running into an error that said `--production=false` is not a flag. After changing `TRUE` -> `true` it went away. Not sure what the best way to test this change though.

Co-authored-by: Simon Gagnon <git@simoncrypta.dev>
Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixture-ok Override the test project fixture check release:fix This PR is a fix
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants