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.

Sign up for GitHub

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

Jump to bottom

Add TS #94

Merged
merged 15 commits into from
Apr 7, 2024
Merged

Add TS #94

merged 15 commits into from
Apr 7, 2024

Conversation

AlbertoFdzM
Copy link
Owner

@AlbertoFdzM AlbertoFdzM commented Apr 7, 2024 •
edited
Loading

Added

  • Add logic for apps and routers with no routes to return an empty array.

Dev

  • Add TypeScript configuration and transpilation scripts.
  • Add types.

AlbertoFdzM and others added 15 commits July 29, 2021 23:20
@AlbertoFdzM
chore(gitignore): add dist to gitignore
210bea2
@AlbertoFdzM
chore(tsconfig): add tsconfig file
1e1ee9b
@AlbertoFdzM
chore(deps): add typescript as devDependency
5ccf749
@AlbertoFdzM
fix(index): fix regexp being passed to RegExp.exec function
616c520
@AlbertoFdzM
Merge branch 'main' of github.com:AlbertoFdzM/express-list-endpoints …
6d98d39 
…into feature/migrate-to-ts

Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
chore(deps): update dev dependencies
e8629a7 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
chore(scripts): update lint npm script and ignore dist directory
1e46fa0 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
chore(ci): remove unnecessary codeql-analysis workflow
1a3be0d 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
chore(ignore): update ignore files
7387a5b 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
chore(package): update package main file
6526890 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
docs(readme): update readme badges and example
2289d64 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
feature(index): add logic for apps with no routes and remove unnecess…
5107b9a 
…ary tests

Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
docs(package): update package info
3e7d623 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
config(ts): update tsconfig.json
4a528c7 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@AlbertoFdzM
docs(index): add types
1b91b3e 
Signed-off-by: Alberto Fernandez <albertofdzm@gmail.com>
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (3b6dd90) to head (1b91b3e).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files 
@@            Coverage Diff            @@
##              main       #94   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           79        80    +1     
  Branches        15        16    +1     
=========================================
+ Hits            79        80    +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AlbertoFdzM AlbertoFdzM merged commit c711938 into main Apr 7, 2024
6 of 7 checks passed
@AlbertoFdzM AlbertoFdzM deleted the feature/migrate-to-ts branch April 7, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AlbertoFdzM @codecov-commenter