Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

UTY-2646: new dotnet version #1443

Merged
merged 24 commits into from
Aug 11, 2020
Merged

UTY-2646: new dotnet version #1443

merged 24 commits into from
Aug 11, 2020

Conversation

BryanJY-Wong
Copy link
Contributor

@BryanJY-Wong BryanJY-Wong commented Jul 30, 2020

Description

Update queue with .NET Core SDK 3.1.302

  • Merge shared-ci and update ci/shared-ci.pinned

Tests

  • Run premerge

Documentation

  • If test pass, update docs

Did you remember a changelog entry?

Primary reviewers

If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

@BryanJY-Wong BryanJY-Wong self-assigned this Jul 30, 2020
@improbable-prow-robot
Copy link

Corresponding JIRA ticket: https://improbableio.atlassian.net/browse/UTY-2646

@improbable-prow-robot improbable-prow-robot added jira/UTY size/XS Denotes a PR that changes 0-14 lines, ignoring generated files. A: maintenance Area: Project maintenance or hygiene A: tooling Area: Tooling size/S Denotes a PR that changes 15-39 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-14 lines, ignoring generated files. labels Jul 30, 2020
ci/sonar-scanner.sh Outdated Show resolved Hide resolved
@@ -23,3 +23,6 @@
# Jetbrains
/Assets/Plugins/Editor/Jetbrains
/Assets/Plugins/Editor/Jetbrains.meta

# Sonarcube dependency
global.json
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't be in the gitignore, and the file won't ever survive a test.sh run

Copy link
Contributor

Choose a reason for hiding this comment

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

Why was this resolved?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I clicked it by mistake when trying to resolve one of the other 2, anyway, I fixed it now

ci/sonar-scanner.sh Show resolved Hide resolved
@BryanJY-Wong BryanJY-Wong force-pushed the feature/new-dotnet-version branch from acde967 to a38e09d Compare August 5, 2020 13:23
Copy link
Contributor

@jamiebrynes7 jamiebrynes7 left a comment

Choose a reason for hiding this comment

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

This needs a change log entry under 'Breaking Changes' since we now require users to install .NET Core 3.1 to run the tooling (codegen & deployment launcher)

CHANGELOG.md Outdated Show resolved Hide resolved
UPGRADE_GUIDE.md Outdated Show resolved Hide resolved
BryanJY-Wong and others added 2 commits August 10, 2020 14:34
@BryanJY-Wong BryanJY-Wong dismissed zeroZshadow’s stale review August 10, 2020 16:49

2/3 changes made, last change not possible

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@BryanJY-Wong BryanJY-Wong merged commit 515adce into develop Aug 11, 2020
@improbable-prow-robot improbable-prow-robot deleted the feature/new-dotnet-version branch August 11, 2020 09:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A: maintenance Area: Project maintenance or hygiene A: tooling Area: Tooling jira/UTY size/S Denotes a PR that changes 15-39 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants