Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

Add corso version using ldflags #1104

Merged
merged 4 commits into from
Oct 14, 2022
Merged

Add corso version using ldflags #1104

merged 4 commits into from
Oct 14, 2022

Conversation

meain
Copy link
Member

@meain meain commented Oct 10, 2022

Description

Makes --version work with actual build version added using ldflags as mentioned in #1088 (comment) .

Waiting for #1088 as this builds on top of that. Currently raised against the branch for that PR.

Type of change

  • 🌻 Feature
  • 🐛 Bugfix
  • 🗺️ Documentation
  • 🤖 Test
  • 💻 CI/Deployment
  • 🐹 Trivial/Minor

Issue(s)

  • #

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Base automatically changed from met-bt to main October 14, 2022 03:37
@meain meain temporarily deployed to Testing October 14, 2022 03:45 Inactive
@meain meain marked this pull request as ready for review October 14, 2022 03:45
@aviator-app
Copy link
Contributor

aviator-app bot commented Oct 14, 2022

Aviator status

Aviator will automatically update this comment as the status of the PR changes.

This PR was merged manually. Merging manually can negatively impact the performance of the queue. Consider using the mergequeue label to merge using Aviator next time.

@meain meain temporarily deployed to Testing October 14, 2022 03:46 Inactive
@meain meain removed the mergequeue label Oct 14, 2022
@meain meain temporarily deployed to Testing October 14, 2022 03:48 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 03:49 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 03:49 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 03:49 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 03:50 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 03:56 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 04:44 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 04:44 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 04:57 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 04:57 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 04:57 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 06:08 Inactive
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@meain meain temporarily deployed to Testing October 14, 2022 06:10 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 06:10 Inactive
@aviator-app aviator-app bot added the blocked Upstream item prevents completion label Oct 14, 2022
@aviator-app
Copy link
Contributor

aviator-app bot commented Oct 14, 2022

PR failed to merge with reason: some CI status(es) failed.
Failed CI(s): Test-Suite

@meain meain temporarily deployed to Testing October 14, 2022 06:26 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 06:26 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 06:26 Inactive
@meain meain temporarily deployed to Testing October 14, 2022 06:26 Inactive
@meain meain added mergequeue and removed blocked Upstream item prevents completion mergequeue labels Oct 14, 2022
@meain meain merged commit 69a6fd1 into main Oct 14, 2022
@meain meain deleted the auto-version branch October 14, 2022 06:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants