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

fix(core): app swaggerLib generation should work using dotnet pathScheme #645

Merged
merged 3 commits into from
Mar 10, 2023

Conversation

pleiris
Copy link
Contributor

@pleiris pleiris commented Mar 8, 2023

codegenProject is generated using nrwl/js which applies nx file naming conventions. Codegen/swagger projects are json/ts libraries so they don't really have to follow dotnet conventions. This adds new nxProjectName normalized option in project generator. This filename is not affected by pathScheme and is used for swaggerProject and codegenProject

fixes #627

@nx-cloud
Copy link

nx-cloud bot commented Mar 8, 2023

pleiris added 3 commits March 10, 2023 10:58
codegenProject is generated using nrwl/js which applies nx file naming conventions. Codegen/swagger projects are json/ts libraries so they don't really have to follow dotnet conventions. This adds new nxFileName normalized option in project generator. This filename is not affected by pathScheme and is used for swaggerProject and codegenProject

fixes #627
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@AgentEnder AgentEnder merged commit 6443e32 into nx-dotnet:master Mar 10, 2023
github-actions bot pushed a commit that referenced this pull request Mar 10, 2023
# [1.20.0](v1.19.1...v1.20.0) (2023-03-10)

### Bug Fixes

* **core:** add @trumbitta/nx-plugin-openapi via the Tree ([c2f2781](c2f2781))
* **core:** app generation should work in encapuslated Nx workspaces ([13c0a2c](13c0a2c))
* **core:** app swaggerLib generation should work using dotnet pathScheme ([#645](#645)) ([6443e32](6443e32))
* **core:** check glob for array length ([#647](#647)) ([8033f76](8033f76))
* **core:** handle null npm scope properly ([455199a](455199a))
* **core:** prevent stale cache entries from breaking swagger on dotnet upgrades ([#633](#633)) ([71f6893](71f6893))
* **core:** remove dependency on memfs and properly specify nx-plugin-openapi as optional peer dep ([cf8f8c3](cf8f8c3))
* **core:** remove dependency on workspace package.json ([d95a3ff](d95a3ff))

### Features

* **core:** nx 15.8.0 support ([897e1a0](897e1a0))

Mar 10, 2023, 5:11 PM
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@pleiris pleiris deleted the dev branch March 11, 2023 00:54
@github-actions
Copy link
Contributor

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] application generator fails to scaffold SwaggerLib with pathScheme:dotnet
2 participants