-
Notifications
You must be signed in to change notification settings - Fork 35
merging master into experiimental #351
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
base: Experimental
Are you sure you want to change the base?
Changes from all commits
9f24c0e
31f4c22
c4adb8f
024c31f
4b003f6
f5feacb
55138c3
4cf4501
dd2dc9c
bd28e8a
f31b89d
0404b16
0b311f6
340fc48
029e6c2
eacf42e
ded6d58
fd3ad5c
b248a35
76b16c9
467b39d
764b09f
4565090
2f9e55c
08ac5d1
24d743d
bdc2e6c
912bb75
382cb65
46007b1
a4a3b7b
5df5fc9
a1a4256
89f14a0
a492113
6404585
9fecdeb
f391f84
fcbe170
e9476c0
06b6c60
e07f085
d35e4da
080a8bf
059b816
8aa06d1
11d827f
b9fb6c7
b7bad0b
59a2dfc
1980ba5
2e7731e
2918960
9505a39
7f0de28
ae70796
741c22d
0982acb
31e4c53
ecbbed5
b4b1e3a
2541f6f
320937f
e6c2dce
e331bf9
47f83c7
60c57fb
962fb8e
d2fe695
81cf303
2223cfe
7d34b89
7c43c69
3c0e914
685893a
376509a
7c126f4
0675033
05b471c
842bd95
703e8fb
2c9020e
9f8e55e
5838d6b
eb88572
4f06640
392c2cd
1adbfac
6286e8b
62de22e
ece30ff
97cd35c
a18c12c
c980348
8e915df
e384234
d1115b6
8b9bd96
f211235
e905d47
3bebfcd
d377c2a
7d6b48b
0d65d67
31cc3e9
271cb7a
761e920
c83cb42
c90af0a
02db5c4
10498e3
21a6d91
c617160
65c390e
8102921
f2fd4f7
3854d76
f6ab872
37c0277
285bdf2
9d4c159
7f58dc4
8e155ce
29afc96
0f32669
9c232a9
9927686
a7a013d
89f3e2c
7a9f539
0775ba0
2d894bd
d98ed0c
bb7481b
0f69668
54d065a
725cb1f
ad69723
32d32b6
da7717b
7bad493
4df1863
f9dfff3
57d9611
ff98548
08a882c
913e73d
8d31b54
8badf8b
c8868ab
10db08d
22ce59c
7fb9e91
3d92c45
5b46af8
ebc17c3
56d8eba
0a5b909
5a0034a
628029a
d62ecc1
a8c5fa1
157b3d3
1c025e2
a5552e5
363b940
0af58be
4783f8a
dd76e6b
74fb9c8
f676c63
e17e742
6221b47
ab0d456
1ed2012
605ab54
9efff2a
46ae8cc
356c079
351bedb
652c449
655dacb
b42369d
85f49a4
d40fa35
d4791fc
df02fcb
aa1de93
0491dbf
010e092
9dccda7
209eb9c
c5f5581
0fb162f
cd367b1
ee17887
07fc268
e78e743
e0aeaeb
5a35180
a31c318
7537545
d80669b
461e83b
05a069e
530733d
5a025b6
779032b
9af4470
d0a39a2
a23a86c
d2a4f1b
c84baa5
eb9ce58
3e93593
c087188
7ad2df8
6eac500
31ea975
f1717eb
f065ac6
88b2e55
6d7bec1
88929ec
7d203f3
38733e1
d05c352
903f245
56ece2d
f54e402
6766c4d
68fb5aa
e97099d
39aca74
05ab70e
dbc48bc
1dc0518
4c22720
2c5a6bb
afa2bca
3b70c6f
297763a
be6131a
dd86421
becf01e
0c306f5
8cb94d0
daf8601
49d7d97
9258aad
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| { | ||
| "repo_notes": [ | ||
| { | ||
| "content": "Use api.oasisweb4.one (not .com) as the canonical base URL for the hosted OASIS API in all examples, code snippets, and documentation. Base URL: https://api.oasisweb4.one, API prefix: https://api.oasisweb4.one/api, Swagger: https://api.oasisweb4.one/swagger/index.html.\n\nIn the Getting Started / Developer Guides section, document both paths clearly:\n1. Using the remote API: Users can call the hosted API at https://api.oasisweb4.one without cloning or running the repo. Include: register (POST /api/avatar/register), verify-email, authenticate (POST /api/avatar/authenticate), then use the JWT in Authorization header for other endpoints. No .NET SDK or local setup required.\n2. Running locally: Clone, restore, build, and run the WebAPI project for local development.", | ||
| "author": "OASIS docs" | ||
| } | ||
| ], | ||
| "pages": [] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -137,6 +137,31 @@ jobs: | |
| dotnet test "STAR ODK/NextGenSoftware.OASIS.STAR.WebAPI.IntegrationTests/NextGenSoftware.OASIS.STAR.WebAPI.IntegrationTests.csproj" --no-build --verbosity normal | ||
| dotnet test "STAR ODK/NextGenSoftware.OASIS.STAR.WebUI.IntegrationTests/NextGenSoftware.OASIS.STAR.WebUI.IntegrationTests.csproj" --no-build --verbosity normal | ||
|
|
||
| # STARAPIClient Unit + Integration + Harness | ||
| test-starapi-client: | ||
| name: Test STARAPIClient | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup .NET | ||
| uses: actions/setup-dotnet@v4 | ||
| with: | ||
| dotnet-version: ${{ env.DOTNET_VERSION }} | ||
|
|
||
| - name: Run STARAPIClient test suite | ||
| shell: pwsh | ||
| run: | | ||
| & "OASIS Omniverse/STARAPIClient/run_star_api_test_suite.ps1" -Configuration Release -KillStaleTestHosts $true | ||
|
|
||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. STARAPIClient tests likely fail on UbuntuHigh Severity The new Additional Locations (1) |
||
| - name: Upload STARAPIClient test artifacts | ||
| uses: actions/upload-artifact@v4 | ||
| if: always() | ||
| with: | ||
| name: starapiclient-test-results | ||
| path: OASIS Omniverse/STARAPIClient/TestResults/ | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. CI artifact upload path with spaces needs quotingMedium Severity The There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Artifact upload path with spaces needs quotingMedium Severity The |
||
| retention-days: 30 | ||
|
|
||
| # Unity Tests (if Unity is available) | ||
| unity-tests: | ||
| name: Unity Tests | ||
|
|
@@ -160,7 +185,7 @@ jobs: | |
| build-and-package: | ||
| name: Build and Package | ||
| runs-on: ubuntu-latest | ||
| needs: [test-oasis-architecture, test-onode, test-star-odk, integration-tests] | ||
| needs: [test-oasis-architecture, test-onode, test-star-odk, integration-tests, test-starapi-client] | ||
| if: github.ref == 'refs/heads/main' || github.ref == 'refs/heads/master' | ||
|
|
||
| steps: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,65 @@ | ||
| # Docs for the Azure Web Apps Deploy action: https://github.com/Azure/webapps-deploy | ||
| # More GitHub Actions for Azure: https://github.com/Azure/actions | ||
|
|
||
| name: Build and deploy ASP.Net Core app to Azure Web App - OASISAPIONODE | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - master | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: windows-latest | ||
| permissions: | ||
| contents: read #This is required for actions/checkout | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Set up .NET Core | ||
| uses: actions/setup-dotnet@v4 | ||
| with: | ||
| dotnet-version: '10.x' | ||
|
|
||
| - name: Build with dotnet | ||
| run: dotnet build --configuration Release | ||
|
|
||
| - name: dotnet publish | ||
| run: dotnet publish -c Release -o "${{env.DOTNET_ROOT}}/myapp" | ||
|
|
||
| - name: Upload artifact for deployment job | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: .net-app | ||
| path: ${{env.DOTNET_ROOT}}/myapp | ||
|
|
||
| deploy: | ||
| runs-on: windows-latest | ||
| needs: build | ||
| permissions: | ||
| id-token: write #This is required for requesting the JWT | ||
| contents: read #This is required for actions/checkout | ||
|
|
||
| steps: | ||
| - name: Download artifact from build job | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| name: .net-app | ||
|
|
||
| - name: Login to Azure | ||
| uses: azure/login@v2 | ||
| with: | ||
| client-id: ${{ secrets.AZUREAPPSERVICE_CLIENTID_6E57E8EB6CA44C78B5CE4C7C568A0523 }} | ||
| tenant-id: ${{ secrets.AZUREAPPSERVICE_TENANTID_BE062CE2CBC344DFA6587A6106793B05 }} | ||
| subscription-id: ${{ secrets.AZUREAPPSERVICE_SUBSCRIPTIONID_AD923DCFBB8745FE900FC743D8F58F66 }} | ||
|
|
||
| - name: Deploy to Azure Web App | ||
| id: deploy-to-webapp | ||
| uses: azure/webapps-deploy@v3 | ||
| with: | ||
| app-name: 'OASISAPIONODE' | ||
| slot-name: 'Production' | ||
| package: . | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| name: Publish NuGet Packages | ||
|
|
||
| on: | ||
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| publish: | ||
| runs-on: windows-latest | ||
| permissions: | ||
| id-token: write | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup .NET | ||
| uses: actions/setup-dotnet@v4 | ||
| with: | ||
| dotnet-version: '10.x' | ||
|
|
||
| - name: Pack all packages | ||
| shell: pwsh | ||
| run: .\publish-nuget.ps1 -SkipPush | ||
|
|
||
| - name: Get NuGet token via Trusted Publishing | ||
| id: get-token | ||
| shell: pwsh | ||
| run: | | ||
| $response = Invoke-RestMethod ` | ||
| -Uri "$env:ACTIONS_ID_TOKEN_REQUEST_URL&audience=api.nuget.org" ` | ||
| -Headers @{ Authorization = "Bearer $env:ACTIONS_ID_TOKEN_REQUEST_TOKEN" } | ||
| echo "NUGET_TOKEN=$($response.value)" >> $env:GITHUB_OUTPUT | ||
|
|
||
| - name: Push all packages to NuGet.org | ||
| shell: pwsh | ||
| run: | | ||
| Get-ChildItem -Path .\nupkgs\*.nupkg | ForEach-Object { | ||
| Write-Host "Pushing $($_.Name)..." | ||
| dotnet nuget push $_.FullName ` | ||
| --source https://api.nuget.org/v3/index.json ` | ||
| --api-key ${{ steps.get-token.outputs.NUGET_TOKEN }} ` | ||
| --skip-duplicate | ||
| } |


Uh oh!
There was an error while loading. Please reload this page.