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(nativeFederation): fixes the missing behaviour of the param outputPath #691

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
57 changes: 57 additions & 0 deletions .github/ISSUE_TEMPLATE/bug-report.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
name: Bug Report
description: Report a bug or regression in functionality

body:
- type: dropdown
id: affected-packages
attributes:
label: With what library do you have an issue?
options:
- native-federation
- module-federation
validations:
required: true

- type: textarea
id: reproduction
attributes:
label: Reproduction of the bug/regression with instructions
description: If on Native Federation, use our [Stackblitz template](https://stackblitz.com/github/rainerhahnekamp/native-federation-stackblitz?file=projects%2Fhost%2Fsrc%2Fapp%2Fapp.routes.ts) to reproduce the issue
placeholder: If the bug/regression does not include a reproduction via StackBlitz or GitHub repo, your issue may be closed without resolution.
validations:
required: true

- type: textarea
id: expected_behavior
attributes:
label: Expected behavior
description: Describe what the expected behavior would be.
validations:
required: true

- type: textarea
id: version
attributes:
label: Versions of Native/Module Federation, Angular, Node, Browser, and operating system
placeholder: |
Native/Module Federation:
Angular:
Node:
Browser:
Operating system(s):
validations:
required: true

- type: textarea
id: other
attributes:
label: Other information

- type: checkboxes
id: assistance
attributes:
label: I would be willing to submit a PR to fix this issue
description: Assistance is provided if you need help submitting a pull request
options:
- label: 'Yes'
- label: 'No'
37 changes: 37 additions & 0 deletions .github/ISSUE_TEMPLATE/feature-request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
name: Feature Request
description: Submit a Request For Consideration

body:
- type: dropdown
id: affected-packages
attributes:
label: For which library do you have a feature request?
options:
- native-federation
- module-federation
- other
multiple: true
validations:
required: true

- type: textarea
id: information
attributes:
label: Information
description: What feature would you like to see added?
validations:
required: true

- type: textarea
id: alternatives
attributes:
label: Describe any alternatives/workarounds you're currently using

- type: checkboxes
id: assistance
attributes:
label: I would be willing to submit a PR to fix this issue
description: Assistance is provided if you need help submitting a pull request
options:
- label: 'Yes'
- label: 'No'
23 changes: 23 additions & 0 deletions .github/ISSUE_TEMPLATE/question.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: Question / Help
description: Ask a question or request help

body:
- type: dropdown
id: affected-packages
attributes:
label: For which library do you need help?
options:
- native-federation
- module-federation
- other
multiple: true
validations:
required: true

- type: textarea
id: information
attributes:
label: Question
description: What do you need help with?
validations:
required: true
4 changes: 4 additions & 0 deletions libs/native-federation/src/builders/build/builder.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,10 @@ export async function* runBuilder(
options.baseHref = nfOptions.baseHref;
}

if(nfOptions.outputPath){
options.outputPath = nfOptions.outputPath;
}

const rebuildEvents = new RebuildHubs();

const adapter = createAngularBuildAdapter(options, context, rebuildEvents);
Expand Down
1 change: 1 addition & 0 deletions libs/native-federation/src/builders/build/schema.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,6 @@ export interface NfBuilderSchema extends JsonObject {
skipHtmlTransform: boolean;
esmsInitOptions: ESMSInitOptions;
baseHref?: string;
outputPath?: string;
ssr: boolean;
} // eslint-disable-line
3 changes: 3 additions & 0 deletions libs/native-federation/src/builders/build/schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,9 @@
"baseHref": {
"type": "string"
},
"outputPath": {
"type": "string"
},
"esmsInitOptions": {
"type": "object",
"description": "Options for esms-module-shims https://github.com/guybedford/es-module-shims?tab=readme-ov-file#init-options",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,7 @@ async function runEsbuild(
splitting: kind === 'mapping-or-exposed',
platform: 'browser',
format: 'esm',
target: ['esnext'],
target: target,
logLimit: kind === 'shared-package' ? 1 : 0,
plugins: plugins || [
createCompilerPlugin(
Expand Down