Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat: allow multiple module paths in router module#16053

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
kr-78493 wants to merge1 commit intonestjs:master
base:master
Choose a base branch
Loading
fromkr-78493:feat-allow-multiple-module-paths

Conversation

@kr-78493
Copy link

@kr-78493kr-78493 commentedDec 11, 2025•
edited
Loading

The Router module allows defining multiple paths for a module. This enhancement updates the route resolution logic to accommodate multiple paths, ensuring that all specified paths are correctly registered and logged.

Closes#16052

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

The Router module allows defining multiple paths for a module. This enhancementupdates the route resolution logic to accommodate multiple paths, ensuring thatall specified paths are correctly registered and logged.Closesnestjs#16052
@coveralls
Copy link

Pull Request Test Coverage Report forBuild 52e28be4-62ac-4c8b-b842-b656b80f3c58

Details

  • 24 of25(96.0%) changed or added relevant lines in4 files are covered.
  • 38 unchanged lines in2 files lost coverage.
  • Overall coverage decreased (-0.5%) to88.867%
Changes Missing CoverageCovered LinesChanged/Added Lines%
packages/common/utils/shared.utils.ts3475.0%
Files with Coverage ReductionNew Missed Lines%
packages/common/file-stream/streamable-file.ts1358.82%
packages/core/injector/topology-tree/tree-node.ts2513.79%
TotalsCoverage Status
Change from baseBuild 1f224c4a-630b-4e8a-9967-d5934153bb78:-0.5%
Covered Lines:7344
Relevant Lines:8264
💛 -Coveralls

Sign up for freeto 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.

Allow multiple module paths on module Router

2 participants

@kr-78493@coveralls
[8]ページ先頭
©2009-2025 Movatter.jp