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

Clean-up aggregate report uploads. #7606

Merged
merged 1 commit into from
Jan 23, 2020
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 0 additions & 26 deletions eng/pipelines/templates/jobs/archetype-sdk-client.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,24 +150,11 @@ jobs:

# We `install` the code quality reports tooling into our local m2 cache separately from building the Maven project
# reports. This means it is available as part of that, but also so that this is not documented in the project report.

- template: ../steps/install-reporting-tools.yml
parameters:
Options: --batch-mode
MavenOptions: $(MemoryOptions) $(LoggingOptions)

# - task: Maven@3
# displayName: 'Install azure-sdk-parent'
# inputs:
# mavenPomFile: parent/pom.xml
# options: '$(DefaultOptions) -DskipTests -Dgpg.skip'
# mavenOptions: '$(MemoryOptions) $(LoggingOptions)'
# javaHomeOption: 'JDKVersion'
# jdkVersionOption: '1.11'
# jdkArchitectureOption: 'x64'
# publishJUnitResults: false
# goals: 'install'

- task: Maven@3
displayName: 'Install Module and Run SpotBugs, Checkstyle, RevApi, and Javadoc'
inputs:
Expand All @@ -180,19 +167,6 @@ jobs:
publishJUnitResults: false
goals: 'install'

# We only copy artifacts into the staging in nightly builds, we don't bother with this for PR validation builds.
- pwsh: |
copy -r target/staging $(Build.ArtifactStagingDirectory)
copy eng/code-quality-reports/src/main/resources/index.html $(Build.ArtifactStagingDirectory)
copy output/dependencies.html (Join-Path $(Build.ArtifactStagingDirectory) "staging")
displayName: 'Copy reports to artifact staging'
condition: and(succeeded(), in(variables['Build.Reason'], 'Schedule', 'Manual'))

- publish: $(Build.ArtifactStagingDirectory)
condition: succeededOrFailed()
mitchdenny marked this conversation as resolved.
Show resolved Hide resolved
displayName: 'Publish Report Artifacts'
artifact: reports

- job: 'Test'
condition: and(succeeded(), ne(variables['Skip.Test'], 'true'))

Expand Down