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

refactor(tsconfig): Make paths scoped #134

Merged
merged 2 commits into from
Oct 5, 2023
Merged

Conversation

heliocastro
Copy link
Contributor

No description provided.

@heliocastro heliocastro requested a review from hoangnt2 October 4, 2023 19:42
@heliocastro heliocastro self-assigned this Oct 4, 2023
@heliocastro heliocastro force-pushed the heliocastro/tsconfig_paths branch 2 times, most recently from 08423e6 to 5fb3ab3 Compare October 5, 2023 06:44
@heliocastro heliocastro changed the title refactor(tsconfig): Make paths scoped and add sw360-frontend path refactor(tsconfig): Make paths scoped Oct 5, 2023
@heliocastro heliocastro force-pushed the heliocastro/tsconfig_paths branch from 5fb3ab3 to bc3b67f Compare October 5, 2023 06:48
- Restrict amount of paths that can be used
- Add next-sw360 as component path for the project, following
next-* style

Signed-off-by: Helio Chissini de Castro <[email protected]>
@heliocastro heliocastro force-pushed the heliocastro/tsconfig_paths branch from bc3b67f to b1a7fed Compare October 5, 2023 06:51
@heliocastro heliocastro requested review from ag4ums, KoukiHama and hoangnt2 and removed request for hoangnt2 October 5, 2023 06:53
Copy link
Contributor

@hoangnt2 hoangnt2 left a comment

Choose a reason for hiding this comment

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

Changes look good

@heliocastro heliocastro merged commit 5558e33 into main Oct 5, 2023
@heliocastro heliocastro deleted the heliocastro/tsconfig_paths branch October 5, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants