Skip to content

Commit

Permalink
Merge branch 'main' into feat-add-option-for-modern-template2
Browse files Browse the repository at this point in the history
  • Loading branch information
yufeih authored Mar 1, 2024
2 parents 950bb94 + 71c3262 commit e37b499
Show file tree
Hide file tree
Showing 641 changed files with 24 additions and 147,436 deletions.
29 changes: 1 addition & 28 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,36 +63,9 @@ jobs:
name: dump
path: /tmp/coredump*

snapshot:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [windows-latest, ubuntu-latest]
permissions:
contents: write
steps:
- uses: actions/checkout@v4
with:
lfs: true

- uses: ./.github/actions/build

- run: dotnet build -c Release samples/extensions/build
- run: dotnet test -c Release -f net8.0 --no-build --filter Stage=Snapshot
working-directory: test/docfx.Snapshot.Tests
env:
SNAPSHOT_TEST: true
BUILD_SERVER: ${{ github.event_name != 'pull_request' }}

- uses: stefanzweifel/git-auto-commit-action@v5
if: github.event_name == 'pull_request'
with:
commit_message: 'test(snapshot): update snapshots ${{ github.sha }} for ${{ matrix.os }}'

publish-docs:
if: github.ref == 'refs/heads/main'
needs: [test, snapshot]
needs: [test]
environment:
name: github-pages
url: ${{ steps.deployment.outputs.page_url }}
Expand Down
2 changes: 2 additions & 0 deletions test/docfx.Snapshot.Tests/PercyTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,8 @@ public async Task SeedHtml()
var samplePath = $"{s_samplesDir}/seed";
Clean(samplePath);

Process.Start("dotnet", $"build \"{s_samplesDir}/seed/dotnet/assembly/BuildFromAssembly.csproj\"").WaitForExit();

var docfxPath = Path.GetFullPath(OperatingSystem.IsWindows() ? "docfx.exe" : "docfx");
Assert.Equal(0, Exec(docfxPath, $"metadata {samplePath}/docfx.json"));
Assert.Equal(0, Exec(docfxPath, $"build {samplePath}/docfx.json"));
Expand Down
Loading

0 comments on commit e37b499

Please sign in to comment.