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

Unify most of our CI under github actions #5371

Merged
merged 20 commits into from
Mar 2, 2021
Merged

Conversation

Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Mar 1, 2021

Unifies our CI under one organization.

  • Move integration builds to github actions

    • Utilizes caching so we builds can benefit from already downloaded Elasticsearch and zips.
    • Easier test matrix.
    • Tests are only reported if they fail
  • Move canary build away from AppVeyor (and Azure devops)

    • Now publishes prereleases on builds on integration branches
    • Move to common .NET feed on feedz.io
  • Stale docs and unit tests are now their own seperate jobs.

  • Only test one random overload instead of all 4.

Moves the general build to utilize nuget lock files, this increases restore times locally and on CI.
- We now also cache nuget packages on ci based on the hash of all lock files.

@Mpdreamz Mpdreamz marked this pull request as ready for review March 1, 2021 17:51
Copy link
Contributor

@stevejgordon stevejgordon left a comment

Choose a reason for hiding this comment

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

LGTM! Great stuff!

Once merged and backported, I can rebase a few of the 7.12 feature PRs onto this to give it some real changes to test?

@Mpdreamz Mpdreamz merged commit 88307bb into 7.x Mar 2, 2021
@Mpdreamz Mpdreamz deleted the fix/7.x/move-to-github-actions branch March 2, 2021 08:38
@Mpdreamz
Copy link
Member Author

Mpdreamz commented Mar 2, 2021

@stevejgordon backported to 7.12 will work on a manual backport to master now :)

Mpdreamz added a commit that referenced this pull request Mar 2, 2021
Mpdreamz added a commit that referenced this pull request Mar 24, 2021
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.

2 participants