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

feat: Add GitRevision with GetValWithKey #246

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

danielfbm
Copy link
Contributor

@danielfbm danielfbm commented Aug 17, 2022

Changes

  • Add GitRevision in apis/meta/v1alpha1 struct to store Git Revision data
  • Incorporate revision data to BuildRunGitStatus which will affect BuildRun.Status

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

Release Notes

feat: Adds revision data to BuildRunGitStatus which will affect BuildRun.Status

@danielfbm danielfbm force-pushed the feat/add-git-revision-struct branch from e568a84 to 150a654 Compare August 17, 2022 12:46
@danielfbm danielfbm changed the title feat: Add GitRevision with GetValWithKey WIP feat: Add GitRevision with GetValWithKey Aug 17, 2022
@alaudabot
Copy link
Contributor

alaudabot commented Aug 17, 2022

Quality Gate

updated at Thu Aug 18 01:11:48 UTC 2022

Status: OK 🟢

Metric Rating
🐛 Bugs: 0 🏗 Reliability: A 🟢
🤮 Code smells: 0 🛠 Maintainability: A 🟢
⚠️ Vulnerabilities: 0 🦺 Security: A 🟢
🔥 Security hotspots: 0 🔎 Security Review: A 🟢
Metric Change set Target (main) After
🧪 Code coverage 100.00%
Coverage on 27 new lines
40.10% 40.30% ⬆️ 🟢
estimated after merge
allowed difference 0
👥 Duplications 0.00%
Duplications on 0 new lines
0.20% 0.20% ↔️ 🟢
estimated after merge
allowed difference 0

[246]

@danielfbm danielfbm force-pushed the feat/add-git-revision-struct branch from 150a654 to 572c2ba Compare August 17, 2022 13:40
l-qing
l-qing previously approved these changes Aug 17, 2022
To provide more complete revision details and simplify usage over
different scenarios, the GitRevision struct is added into `apis/meta/v1alpha1`
and added into BuildRunGitStatus with the GetValWithKey implementation
making the values also available for variable substitution scenarios
@danielfbm danielfbm force-pushed the feat/add-git-revision-struct branch from 04c5307 to e8fb123 Compare August 18, 2022 01:06
@danielfbm danielfbm merged commit ffbea87 into main Aug 18, 2022
@danielfbm danielfbm deleted the feat/add-git-revision-struct branch August 18, 2022 01:13
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