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

AppCenterDistributeV3: Use newer version of the "appcenter-file-upload-client-node" #15604

Conversation

ondrej-merkun-ms
Copy link
Contributor

@ondrej-merkun-ms ondrej-merkun-ms commented Dec 14, 2021

Task name: AppCenterDistributeV3

Description: Increase the version used of package "appcenter-file-upload-client-node" to version that already has content-type for aab. This is a needed fix for aab distribution in AppCenter.
Also add "skipLibCheck" to our task's tsconfig, as that's the setup that we also use in appcenter-cli and without it, we would be getting TS errors for our internal appcenter-file-upload-client-node package, probably because of many conflicting package versions, such as of TS, node, npm, ... between the packages.

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) N

Attached related issue: (Y/N) N/A

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

…d-client-node" package which contains content-type declaration for aab files
@lucen-ms lucen-ms closed this Dec 15, 2021
@lucen-ms lucen-ms reopened this Dec 15, 2021
@lucen-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

… appcenter-file-upload-client-node and it's dependencies in hopes of fixing the Azure pipeline
@ondrej-merkun-ms
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 15604 in repo microsoft/azure-pipelines-tasks

@lucen-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@lucen-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@lucen-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@andrii-z4i
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 15604 in repo microsoft/azure-pipelines-tasks

@lucen-ms
Copy link
Contributor

lucen-ms commented Jan 3, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

Tasks/AppCenterDistributeV3/task.loc.json Outdated Show resolved Hide resolved
Tasks/AppCenterDistributeV3/tsconfig.json Show resolved Hide resolved
@lucen-ms lucen-ms merged commit 1ec191f into microsoft:master Jan 4, 2022
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.

3 participants