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

feat: Heroku updates: #135

Merged
merged 4 commits into from
May 11, 2021
Merged

feat: Heroku updates: #135

merged 4 commits into from
May 11, 2021

Conversation

mthomps4
Copy link
Contributor

@mthomps4 mthomps4 commented May 7, 2021

Changes

Heroku updates:

  • Now supports apollo-server-micro over apollo-server-express
  • adds Procfile on creation
  • adds .nvmrc
  • fixes typo in package.json
  • updates package.json conditionals (removes apollo-server-express)
  • fixes named export bug in test/helpers.ts

Node Version:

  • Removes Node v10 support (dead)
  • Attempt to run v16 (needs some love before we can adopt fully)
  • TODO: Fix E2E tests with app ENVs for local dev

Checklist

  • Generating a new app works

- Now supports
- adds procfile creation
- adds .nvmrc
- fixes typo in package.json
- updates package.json conditionals (removes server-express)
- fixes named export bug in test/helpers.ts
@mthomps4 mthomps4 self-assigned this May 7, 2021
@mthomps4 mthomps4 changed the title Heroku updates: feat: Heroku updates: May 7, 2021
Copy link
Member

@cball cball left a comment

Choose a reason for hiding this comment

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

Looks great. 👍

@@ -66,6 +66,8 @@ module.exports = async ({ name, ...answers }) => {
variables.host.createAppsAndPipelines,
task: async () => {
const repoName = variables.githubRepo.match(/(\w+\/\w+).git$/)[1];
// ! Bug w/ Pipelines, better error handling needed.
// await fsPromises.writeFile("Procfile", "web: yarn db:deploy && yarn start");
Copy link
Member

Choose a reason for hiding this comment

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

Should we add a follow up issue to track this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll get this added as a follow-up.

@mthomps4 mthomps4 merged commit f2d657c into canary May 11, 2021
@mthomps4 mthomps4 deleted the mt-heroku-setup-pt-1 branch May 11, 2021 13:10
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