Skip to content

Commit 28edf60

Browse files
authored
Remove channel name concept from update-dependencies (#4250)
1 parent 3f6b898 commit 28edf60

File tree

4 files changed

+4
-8
lines changed

4 files changed

+4
-8
lines changed

CONTRIBUTING.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,10 @@ The following examples illustrate how to run `update-dependencies`:
8282
> ./eng/Set-DotnetVersions.ps1 -ProductVersion 3.1 -SdkVersion 3.1.302 -RuntimeVersion 3.1.6 -AspnetVersion 3.1.6
8383
```
8484

85-
- Update the .NET Monitor version
85+
- Update the .NET Monitor version (uses a helper script for running update-dependencies)
8686

8787
``` console
88-
> dotnet run --project .\eng\update-dependencies\update-dependencies.csproj 6.0 --product-version monitor=6.0.0-rc.1.21515.7 --channel-name 6.0/rc.1
88+
> ./eng/Set-DotnetVersions.ps1 -ProductVersion 6.0 -MonitorVersion 6.0.0-rc.1.21515.7
8989
```
9090

9191
- Update the PowerShell version used in the 6.0 images

eng/pipelines/update-dependencies-monitor.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ stages:
3131
- pwsh: $(engPath)/Get-MonitorDropVersions.ps1 -Channel $(MajorMinorVersion)/$(ChannelQuality)
3232
displayName: Get Versions
3333
- powershell: |
34-
$customArgs = "$(MajorMinorVersion) --product-version monitor=$(monitorVer) --channel-name $(MajorMinorVersion)/$(ChannelQuality) --version-source-name dotnet/dotnet-monitor/$(MajorMinorVersion) --stable-branding=$(stableBranding)"
34+
$customArgs = "$(MajorMinorVersion) --product-version monitor=$(monitorVer) --version-source-name dotnet/dotnet-monitor/$(MajorMinorVersion) --stable-branding=$(stableBranding)"
3535
3636
$customArgsArray = @($customArgs)
3737
echo "##vso[task.setvariable variable=customArgsArray]$($customArgsArray | ConvertTo-Json -Compress -AsArray)"

eng/update-dependencies/DockerfileShaUpdater.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,6 @@ public static IEnumerable<IDependencyUpdater> CreateUpdaters(
208208
.Replace("$ARCHIVE_EXT", archiveExt)
209209
.Replace("$VERSION_DIR", versionDir)
210210
.Replace("$VERSION_FILE", versionFile)
211-
.Replace("$CHANNEL_NAME", _options.ChannelName)
212211
.Replace("$OS", _os)
213212
.Replace("$OPTIONAL_OS", optionalOs)
214213
.Replace("$ARCH", _arch)

eng/update-dependencies/Options.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ public class Options
1414
public string ChecksumSasQueryString { get; }
1515
public bool ComputeChecksums { get; }
1616
public string DockerfileVersion { get; }
17-
public string ChannelName { get; }
1817
public string Email { get; }
1918
public string Password { get; }
2019
public string GitHubProject => "dotnet-docker";
@@ -31,14 +30,13 @@ public class Options
3130
public bool UpdateOnly => Email == null || Password == null || User == null || TargetBranch == null;
3231
public bool IsInternal => !string.IsNullOrEmpty(BinarySasQueryString) || !string.IsNullOrEmpty(ChecksumSasQueryString);
3332

34-
public Options(string dockerfileVersion, string[] productVersion, string channelName, string versionSourceName, string email, string password, string user,
33+
public Options(string dockerfileVersion, string[] productVersion, string versionSourceName, string email, string password, string user,
3534
bool computeShas, bool stableBranding, string binarySas, string checksumSas, string sourceBranch, string targetBranch, string org, string project, string repo)
3635
{
3736
DockerfileVersion = dockerfileVersion;
3837
ProductVersions = productVersion
3938
.Select(pair => pair.Split(new char[] { '=' }, 2))
4039
.ToDictionary(split => split[0].ToLower(), split => split.Skip(1).FirstOrDefault());
41-
ChannelName = channelName;
4240
VersionSourceName = versionSourceName;
4341
Email = email;
4442
Password = password;
@@ -72,7 +70,6 @@ public static IEnumerable<Symbol> GetCliSymbols() =>
7270
{
7371
new Argument<string>("dockerfile-version", "Version of the Dockerfiles to update"),
7472
new Option<string[]>("--product-version", "Product versions to update (<product-name>=<version>)"),
75-
new Option<string>("--channel-name", "The name of the channel from which to find product files."),
7673
new Option<string>("--version-source-name", "The name of the source from which the version information was acquired."),
7774
new Option<string>("--email", "GitHub or AzDO email used to make PR (if not specified, a PR will not be created)"),
7875
new Option<string>("--password", "GitHub or AzDO password used to make PR (if not specified, a PR will not be created)"),

0 commit comments

Comments
 (0)