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

Represent VCS properties #108

Closed
lgolding opened this Issue Feb 27, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@lgolding
Contributor

lgolding commented Feb 27, 2018

We need a way to say something like “This tool was run against commit 056eefc of repo https://github.com/example/repo_1.” That would be fairly easy to add, but it might be tricky to express it in a way that is independent of the VCS (Git vs VSTS version control vs Subversion…).

@michaelcfanning

This comment has been minimized.

Show comment
Hide comment
@michaelcfanning

michaelcfanning Apr 13, 2018

Contributor

My hat in the ring proposal, add a new run.versionControl property. These should minimally be considered.

versionControl

  • url
  • branch
  • revisionId
  • timestamp (for vc systems that can sync by time)
  • label (for vs systems that can sync by label)
Contributor

michaelcfanning commented Apr 13, 2018

My hat in the ring proposal, add a new run.versionControl property. These should minimally be considered.

versionControl

  • url
  • branch
  • revisionId
  • timestamp (for vc systems that can sync by time)
  • label (for vs systems that can sync by label)
@lgolding

This comment has been minimized.

Show comment
Hide comment
@lgolding

lgolding Apr 13, 2018

Contributor

@michaelcfanning This looks good to me.

I think we need to bring this into CSD.1, given your customer's needs.

Contributor

lgolding commented Apr 13, 2018

@michaelcfanning This looks good to me.

I think we need to bring this into CSD.1, given your customer's needs.

@lgolding lgolding self-assigned this Apr 15, 2018

@lgolding lgolding added CSD.1 and removed CSD.2 labels Apr 15, 2018

lgolding added a commit that referenced this issue Apr 16, 2018

lgolding added a commit that referenced this issue Apr 18, 2018

@lgolding

This comment has been minimized.

Show comment
Hide comment
@lgolding

lgolding Apr 18, 2018

Contributor

Closed by 4cca601.

Contributor

lgolding commented Apr 18, 2018

Closed by 4cca601.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment