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

Run engine tests #892

Merged
merged 21 commits into from May 10, 2021
Merged

Run engine tests #892

merged 21 commits into from May 10, 2021

Conversation

sebastienros
Copy link
Owner

No description provided.

@lahma
Copy link
Collaborator

lahma commented May 8, 2021

Windows, Linux and MacOS will make a lovely combo 👌🏻

@sebastienros
Copy link
Owner Author

Focusing on Linux only for now, will add the matrix when it works. Though not sure it should run everything, will depend on the time it takes. Or maybe just for the Publish step.

@lahma
Copy link
Collaborator

lahma commented May 8, 2021

Some build logic probably helps, maybe in c# 🤔

@sebastienros
Copy link
Owner Author

Everything seems to work, even the publish task using the tags (from any branch). I just used a fake API key to test.

@sebastienros sebastienros requested a review from lahma May 8, 2021 19:30
@@ -1,6 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<TargetFrameworks>net461;net5.0</TargetFrameworks>
<TargetFrameworks>net5.0</TargetFrameworks>
Copy link
Collaborator

Choose a reason for hiding this comment

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

would be nice to have windows included and also net461 tested

Copy link
Collaborator

@lahma lahma left a comment

Choose a reason for hiding this comment

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

Looking good. Would be nice to have the build matrix setup for windows and macos too. I guess with this we won't easily get the release notes generation based on commits and contributors document generation, but maybe just basic yaml then instead of new era build automation 😛

@sebastienros
Copy link
Owner Author

Added windows builds, running met461 only on it since 5.0 is tested on linux. That keeps the PR builds under 5 minutes.
Also not running on windows for publication.

@sebastienros
Copy link
Owner Author

Re-added net5.0 on windows

@sebastienros sebastienros merged commit 4d83ebd into dev May 10, 2021
@sebastienros sebastienros deleted the sebros/tests branch May 10, 2021 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants