Skip to content

feat(templates/deno): deploying to Deno Deploy#2824

Merged
pcattori merged 4 commits intomainfrom
pedro/deno-deploy
May 21, 2022
Merged

feat(templates/deno): deploying to Deno Deploy#2824
pcattori merged 4 commits intomainfrom
pedro/deno-deploy

Conversation

@pcattori
Copy link
Copy Markdown
Contributor

@pcattori pcattori commented Apr 15, 2022

Testing Strategy

Deployment test for Deno Deploy.

Tried the test locally by using my own Deno Deploy project name, and that deployed successfully. (https://salty-bee-35.deno.dev/)

@pcattori pcattori force-pushed the pedro/deno-deploy branch 3 times, most recently from 7da0791 to 43be45f Compare April 15, 2022 22:45
@pcattori pcattori requested review from jacob-ebey and mjackson April 15, 2022 22:47
@pcattori pcattori force-pushed the pedro/deno-deploy branch from 43be45f to d33cd7a Compare April 15, 2022 22:52
@pcattori pcattori marked this pull request as ready for review April 15, 2022 22:57
@pcattori pcattori force-pushed the pedro/deno-deploy branch 2 times, most recently from 343cc17 to f64386d Compare May 19, 2022 18:28
@MichaelDeBoey MichaelDeBoey changed the title feat(templates/deno-deploy): add new deno deploy template feat(templates/deno): add Deno Deploy instructions May 19, 2022
@pcattori pcattori changed the title feat(templates/deno): add Deno Deploy instructions feat(templates/deno): instructions for deploying to Deno Deploy May 19, 2022
@pcattori pcattori force-pushed the pedro/deno-deploy branch from 7f76c22 to 335d7bd Compare May 19, 2022 18:34
@pcattori pcattori requested review from MichaelDeBoey and removed request for jacob-ebey May 19, 2022 19:12
@pcattori pcattori force-pushed the pedro/deno-deploy branch from 335d7bd to 84f08a8 Compare May 19, 2022 20:52
@pcattori pcattori changed the title feat(templates/deno): instructions for deploying to Deno Deploy feat(templates/deno): deploying to Deno Deploy May 20, 2022
@pcattori pcattori force-pushed the pedro/deno-deploy branch from e13fcc9 to 84def56 Compare May 21, 2022 14:09
@pcattori pcattori force-pushed the pedro/deno-deploy branch from 84def56 to 9975428 Compare May 21, 2022 19:19
Copy link
Copy Markdown
Contributor

@mcansh mcansh left a comment

Choose a reason for hiding this comment

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

LGTM 🦕☄️

@pcattori pcattori merged commit a3b3b27 into main May 21, 2022
@pcattori pcattori deleted the pedro/deno-deploy branch May 21, 2022 19:43
eastlondoner pushed a commit to eastlondoner/remix that referenced this pull request Jun 8, 2022
* feat(templates/deno): deploy to deno deploy

* feat(templates/deno): recommend the deno extension for vscode

* docs(templates/deno): explain how to setup type hints within editor

* ci(deno-deploy): add deploy test for deno deploy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants