-
Notifications
You must be signed in to change notification settings - Fork 19
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
Update Akka to 1.4.4 #42
Changes from 14 commits
eb171b1
8b5e467
9408564
965a9dd
90e272b
f401046
2d569c6
95051e6
55138e4
c9f74f9
7ac6668
84fb9d3
b8c8dad
81f0d6d
dabd3f9
2a8743f
34eb8c1
8ba4135
25bce86
fdf091c
dcd4dd5
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 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<configuration> | ||
<packageSources> | ||
<clear /> | ||
<add key="nuget.org" value="https://www.nuget.org/api/v2/" /> | ||
<add key="akka-nightly" value="https://www.myget.org/F/akkadotnet/api/v3/index.json" /> | ||
<add key="netcorenightly" value="https://www.myget.org/F/akkadotnet/api/v3/index.json" /> | ||
</packageSources> | ||
</configuration> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,12 @@ | ||
parameters: | ||
name: '' | ||
displayName: '' | ||
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. Roll back these changes - the current build files are fine and are in-use. |
||
vmImage: '' | ||
scriptFileName: '' | ||
scriptArgs: 'all' | ||
timeoutInMinutes: 120 | ||
|
||
jobs: | ||
- job: ${{ parameters.name }} | ||
displayName: ${{ parameters.displayName }} | ||
timeoutInMinutes: ${{ parameters.timeoutInMinutes }} | ||
pool: | ||
vmImage: ${{ parameters.vmImage }} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
# Pull request validation for Linux against the `dev` and `master` branches | ||
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. Roll back these changes - the current build files are fine and are in-use. |
||
# See https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema for reference | ||
trigger: | ||
branches: | ||
include: | ||
- dev | ||
- master | ||
|
||
name: $(Year:yyyy).$(Month).$(DayOfMonth)$(Rev:.r) | ||
|
||
pr: | ||
autoCancel: true # indicates whether additional pushes to a PR should cancel in-progress runs for the same PR. Defaults to true | ||
branches: | ||
include: [ dev, master ] # branch names which will trigger a build | ||
|
||
jobs: | ||
- template: azure-pipeline.template.yaml | ||
parameters: | ||
name: Ubuntu | ||
vmImage: 'ubuntu-16.04' | ||
scriptFileName: ./build.sh | ||
scriptArgs: all |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,17 +14,9 @@ pr: | |
name: $(Year:yyyy).$(Month).$(DayOfMonth)$(Rev:.r) | ||
|
||
jobs: | ||
- template: azure-pipeline.template.yaml | ||
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. Roll back these changes - the current build files are fine and are in-use. |
||
parameters: | ||
name: 'windows_pr' | ||
displayName: 'Windows PR Validation' | ||
vmImage: 'vs2017-win2016' | ||
scriptFileName: build.cmd | ||
scriptArgs: all | ||
- template: azure-pipeline.template.yaml | ||
parameters: | ||
name: 'linux_pr' | ||
displayName: 'Linux PR Validation' | ||
vmImage: 'ubuntu-16.04' | ||
scriptFileName: ./build.sh | ||
scriptArgs: RunTests | ||
- template: azure-pipeline.template.yaml | ||
parameters: | ||
name: Windows | ||
vmImage: 'vs2017-win2016' | ||
scriptFileName: build.cmd | ||
scriptArgs: all |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,22 +10,23 @@ trigger: | |
include: | ||
- refs/tags/* | ||
|
||
pr: none | ||
|
||
variables: | ||
- group: signingSecrets #create this group with SECRET variables `signingUsername` and `signingPassword` | ||
- group: nugetKeys #create this group with SECRET variables `nugetKey` | ||
- name: githubConnectionName | ||
value: yourConnection #replace this | ||
value: AkkaDotNet_Releases | ||
- name: projectName | ||
value: yourProjectName #replace this | ||
value: Akka.Persistence.SqlServer | ||
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. Need to change this to Akka.Persistence.Redis |
||
- name: githubRepositoryName | ||
value: yourOrganization/yourRepo #replace this | ||
|
||
value: akkadotnet/Akka.Persistence.SqlServer | ||
steps: | ||
- task: BatchScript@1 | ||
displayName: 'FAKE Build' | ||
inputs: | ||
filename: build.cmd | ||
arguments: 'All SignClientUser=$(signingUsername) SignClientSecret=$(signingPassword) nugetpublishurl=https://www.nuget.org/api/v2/package nugetkey=$(nugetKey)' | ||
arguments: 'nuget nugetpublishurl=https://www.nuget.org/api/v2/package nugetkey=$(nugetKey)' | ||
|
||
- task: GitHubRelease@0 | ||
displayName: 'GitHub release (create)' | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,6 +37,7 @@ $NugetVersion = "4.1.0"; | |
$NugetUrl = "https://dist.nuget.org/win-x86-commandline/v$NugetVersion/nuget.exe" | ||
$ProtobufVersion = "3.4.0" | ||
$DocfxVersion = "2.49.0" | ||
$RedisVersion = "3.0.503" | ||
|
||
# Make sure tools folder exists | ||
$PSScriptRoot = Split-Path $MyInvocation.MyCommand.Path -Parent | ||
|
@@ -115,6 +116,18 @@ if (!(Test-Path $FakeExePath)) { | |
} | ||
} | ||
|
||
########################################################################### | ||
# INSTALL REDIS | ||
########################################################################### | ||
$RedisExePath = Join-Path $ToolPath "redis-64/tools/redis-server.exe" | ||
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. Shouldn't need this if we're using Docker.NET |
||
if (!(Test-Path $RedisExePath)) { | ||
Write-Host "Installing Redis-64..." | ||
Invoke-Expression "&`"$NugetPath`" install redis-64 -ExcludeVersion -Version $RedisVersion -OutputDirectory `"$ToolPath`"" | Out-Null; | ||
if ($LASTEXITCODE -ne 0) { | ||
Throw "An error occured while restoring Redis-64 from NuGet." | ||
} | ||
} | ||
|
||
########################################################################### | ||
# Docfx | ||
########################################################################### | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,29 +1,43 @@ | ||
<Project Sdk="Microsoft.NET.Sdk"> | ||
<Import Project="..\common.props" /> | ||
|
||
<PropertyGroup> | ||
<AssemblyTitle>Akka.Persistence.Redis.Tests</AssemblyTitle> | ||
<TargetFrameworks>$(NetFrameworkTestVersion);$(NetCoreTestVersion)</TargetFrameworks> | ||
</PropertyGroup> | ||
|
||
<PropertyGroup Condition="'$(OS)' == 'Windows_NT'"> | ||
<TargetFrameworks>$(NetFrameworkTestVersion);$(NetCoreTestVersion)</TargetFrameworks> | ||
</PropertyGroup> | ||
<TargetFrameworks>$(NetFrameworkTestVersion);$(NetCoreTestVersion)</TargetFrameworks> | ||
</PropertyGroup> | ||
|
||
<!-- disable .NET Framework (Mono) on Linux--> | ||
<PropertyGroup Condition="'$(OS)' != 'Windows_NT'"> | ||
<TargetFrameworks>$(NetCoreTestVersion)</TargetFrameworks> | ||
</PropertyGroup> | ||
<!-- disable .NET Framework (Mono) on Linux--> | ||
<PropertyGroup Condition="'$(OS)' != 'Windows_NT'"> | ||
<TargetFrameworks>$(NetCoreTestVersion)</TargetFrameworks> | ||
</PropertyGroup> | ||
|
||
|
||
<ItemGroup> | ||
<ProjectReference Include="..\Akka.Persistence.Redis\Akka.Persistence.Redis.csproj" /> | ||
</ItemGroup> | ||
|
||
|
||
<ItemGroup> | ||
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="$(TestSdkVersion)" /> | ||
<!-- Needed by Redis to run --> | ||
<PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="4.7.0" /> | ||
<PackageReference Include="Docker.DotNet" Version="$(DockerVersion)" /> | ||
<PackageReference Include="xunit" Version="$(XunitVersion)" /> | ||
<PackageReference Include="xunit.runner.visualstudio" Version="$(XunitVersion)" /> | ||
<PackageReference Include="FluentAssertions" Version="$(FluentAssertionsVersion)" /> | ||
</ItemGroup> | ||
|
||
<ItemGroup> | ||
<PackageReference Include="FluentAssertions" Version="4.19.4" /> | ||
<PackageReference Include="Akka.TestKit" Version="$(AkkaVersion)" /> | ||
<PackageReference Include="Akka.Persistence.TCK" Version="$(AkkaVersion)" /> | ||
<PackageReference Include="Akka.TestKit.Xunit2" Version="$(AkkaVersion)" /> | ||
<PackageReference Include="Akka.Streams.TestKit" Version="$(AkkaVersion)" /> | ||
</ItemGroup> | ||
|
||
<PropertyGroup Condition=" '$(Configuration)' == 'Release' "> | ||
<DefineConstants>$(DefineConstants);RELEASE</DefineConstants> | ||
</PropertyGroup> | ||
</Project> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can get rid of this - don't need it anymore.