Skip to content

Commit 3af4b91

Browse files
author
Zachary Eisinger
authored
Merge pull request #95 from thashepherd/deprecate_version
Deprecate version property
2 parents 4114018 + fe319a6 commit 3af4b91

File tree

6 files changed

+421
-229
lines changed

6 files changed

+421
-229
lines changed

action.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ author: 'GitHub'
44
branding:
55
icon: play
66
color: green
7-
inputs:
7+
inputs:
88
dotnet-version:
99
description: 'SDK version to use. Examples: 2.2.104, 3.1, 3.1.x'
1010
source-url:
@@ -13,10 +13,6 @@ inputs:
1313
description: 'Optional OWNER for using packages from GitHub Package Registry organizations/users other than the current repository''s owner. Only used if a GPR URL is also provided in source-url'
1414
config-file:
1515
description: 'Optional NuGet.config location, if your NuGet.config isn''t located in the root of the repo.'
16-
# Deprecated option, do not use. Will not be supported after October 1, 2019
17-
version:
18-
description: 'Deprecated. Use dotnet-version instead. Will not be supported after October 1, 2019'
19-
deprecationMessage: 'The version property will not be supported after October 1, 2019. Use dotnet-version instead'
2016
runs:
2117
using: 'node12'
2218
main: 'dist/index.js'

dist/index.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7872,13 +7872,13 @@ function run() {
78727872
return __awaiter(this, void 0, void 0, function* () {
78737873
try {
78747874
//
7875-
// Version is optional. If supplied, install / use from the tool cache
7876-
// If not supplied then task is still used to setup proxy, auth, etc...
7875+
// dotnet-version is optional, but needs to be provided for most use cases.
7876+
// If supplied, install / use from the tool cache.
7877+
// If not supplied, look for version in ./global.json.
7878+
// If a valid version still can't be identified, nothing will be installed.
7879+
// Proxy, auth, (etc) are still set up, even if no version is identified
78777880
//
7878-
let version = core.getInput('version');
7879-
if (!version) {
7880-
version = core.getInput('dotnet-version');
7881-
}
7881+
let version = core.getInput('dotnet-version');
78827882
if (!version) {
78837883
// Try to fall back to global.json
78847884
core.debug('No version found, trying to find version from global.json');

0 commit comments

Comments
 (0)