Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

Add GitHub Workflow #1

Closed
wants to merge 3 commits into from
Closed
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
26 changes: 26 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
name: Build

on:
push:
branches: [ master ]
Comment on lines +4 to +5
Copy link

Choose a reason for hiding this comment

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

I guess for this trigger we'd want CD flow and not CI.

Copy link
Author

@ebickle ebickle Sep 28, 2023

Choose a reason for hiding this comment

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

@AleF83 This PR adds only CI, not CD. We'll have to determine where we would want to deploy to (e.g. GitHub Packages? NuGet again?) first. I was hoping we could merge this change to deprecate Appveyor for the repo.

Copy link

Choose a reason for hiding this comment

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

I meant we don't need push trigger.

pull_request:
branches: [ master ]

jobs:
build:
runs-on: windows-latest

steps:
- name: Checkout repository
uses: actions/checkout@v3

- name: Setup .NET Core
uses: actions/setup-dotnet@v3
with:
dotnet-version: 3.0.x

- name: Build
run: dotnet build -c Release

- name: Run unit tests
run: dotnet test test/Soluto.Collections.RadixTree.Tests/Soluto.Collections.RadixTree.Tests.csproj -c Release
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>netcoreapp1.1</TargetFramework>
<TargetFramework>netcoreapp3.0</TargetFramework>
</PropertyGroup>

<ItemGroup>
Expand Down
Loading