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

Breestryker/polichecklatest #550

Merged
merged 2 commits into from
Dec 2, 2021
Merged

Breestryker/polichecklatest #550

merged 2 commits into from
Dec 2, 2021

Conversation

Breanna-Stryker
Copy link
Contributor

Description

Pinned to latest major versions. Changed artifact output as it changed between versions.

There doesn't appear to be a decorator to pin to latest, and you are required to at least specify the major version.
Based on https://docs.microsoft.com/en-us/azure/devops/pipelines/process/tasks?view=azure-devops&tabs=yaml#task-versions

This is still being hosted out of the original devops area as policheck and credential scanner are not available tasks in our new pipeline store yet. I will move them over asap.

Issue reference

The issue this PR will close: #547

Checklist

Please make sure you've completed the relevant tasks for this PR out of the following list:

  • All acceptance criteria in the backlog item are met
  • The documentation is updated to cover any new or changed features
  • Manual tests have passed
  • Relevant issues are linked to this PR

@Breanna-Stryker Breanna-Stryker requested a review from a team December 1, 2021 18:03
Copy link
Contributor

@glennmusa glennmusa left a comment

Choose a reason for hiding this comment

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

looks good thanks!

@glennmusa glennmusa merged commit 2744efe into main Dec 2, 2021
@glennmusa glennmusa deleted the breestryker/polichecklatest branch December 2, 2021 15:19
jwaltireland pushed a commit to ARPA-H/AzureMissionLZ that referenced this pull request Nov 14, 2024
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.

Update policheck version
2 participants