Skip to content

Conversation

@CorentinDoue
Copy link
Contributor

#9445 adds google-nodejs-typescript template but it can't be used because it is not in the recommended-list.
Screenshot from 2021-06-07 10-41-34

This PR allow the usage of the template by adding its name in the recommended-list.
Screenshot from 2021-06-07 10-43-20

@codecov
Copy link

codecov bot commented Jun 7, 2021

Codecov Report

Merging #9557 (f2523c0) into master (ddab624) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9557   +/-   ##
=======================================
  Coverage   86.57%   86.57%           
=======================================
  Files         322      322           
  Lines       12109    12109           
=======================================
  Hits        10483    10483           
  Misses       1626     1626           
Impacted Files Coverage Δ
lib/templates/recommended-list/index.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ddab624...f2523c0. Read the comment docs.

Copy link
Contributor

@pgrzesik pgrzesik left a comment

Choose a reason for hiding this comment

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

Thank you @CorentinDoue, good catch 👍

@pgrzesik pgrzesik changed the title fix(Templates): allow usage of google-nodejs-typescript template fix(Templates): Allow usage of google-nodejs-typescript template Jun 7, 2021
@pgrzesik pgrzesik merged commit accf5bd into serverless:master Jun 7, 2021
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.

2 participants