Skip to content
This repository has been archived by the owner on Jul 3, 2020. It is now read-only.

Added a version controller and used proper ruby gems to gather data. #103

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

aaronlippold
Copy link
Member

I wanted to use rugged here as it is a fully cross platform solution with zero deps but still had to use the git gem due to above FR I put in at rugged.

Fixes #95

Signed-off-by: Aaron Lippold [email protected]

@aaronlippold aaronlippold requested review from robthew and rbclark April 22, 2019 02:58
@aaronlippold aaronlippold force-pushed the disp_ver branch 3 times, most recently from 9689866 to f071566 Compare April 22, 2019 03:16
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 03:25 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 03:40 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 03:44 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 03:48 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 03:58 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 04:02 Failure
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 04:29 Failure
@aaronlippold aaronlippold requested a deployment to mitre-heimdall-pr-103 April 22, 2019 04:38 Abandoned
@aaronlippold aaronlippold had a problem deploying to mitre-heimdall-pr-103 April 22, 2019 04:39 Failure
@aaronlippold aaronlippold requested a deployment to mitre-heimdall-pr-103 April 22, 2019 04:43 Abandoned
TODO: Need to update rake tasks to use the controller on tagged releases.

Signed-off-by: Aaron Lippold <[email protected]>
Signed-off-by: Aaron Lippold <[email protected]>
Signed-off-by: Aaron Lippold <[email protected]>
@aaronlippold aaronlippold temporarily deployed to mitre-heimdall-pr-103 April 22, 2019 04:56 Inactive
@aaronlippold aaronlippold temporarily deployed to mitre-heimdall-pr-103 April 22, 2019 05:15 Inactive
Signed-off-by: Aaron Lippold <[email protected]>
@aaronlippold aaronlippold requested a deployment to mitre-heimdall-pr-103 April 22, 2019 05:23 Abandoned
}

File.open('config/VERSION','w') do |f|
f.write(JSON.pretty_generate(version))
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not quite sure this will work as intended. It actually might be completely impossible to add a version in this manner since adding the version to the hash itself will change what the hash should be. Also, any squashes or merges done on Github will all have the wrong SHA since rails is not run on those. Heroku used to provide a SOURCE_VERSION ENV but is seems they have removed it.

@aaronlippold
Copy link
Member Author

aaronlippold commented Apr 22, 2019 via email

@aaronlippold aaronlippold requested a deployment to mitre-heimdall-pr-103 April 29, 2019 19:17 Abandoned
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Version in footer not available in container
3 participants