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

set version to 1.8.0-dev #137

Merged
merged 1 commit into from
May 10, 2021
Merged

set version to 1.8.0-dev #137

merged 1 commit into from
May 10, 2021

Conversation

aschmahmann
Copy link
Contributor

No description provided.

@aschmahmann aschmahmann marked this pull request as draft April 1, 2021 01:35
@@ -4,7 +4,7 @@ package config
const CurrentAppName = "ipfs-update"

// CurrentVersionNumber is the current application's version literal
const CurrentVersionNumber = "1.7.1"
const CurrentVersionNumber = "1.8.0-dev"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Stebalien does this dev pattern make sense for this binary? I don't have a problem continuing it, but I'd like to better understand what we're getting out of it (e.g. is it better user bug reports, etc.).

Copy link
Member

Choose a reason for hiding this comment

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

It's not critical... But in theory, it means we can get better version info.

@Stebalien Stebalien marked this pull request as ready for review May 10, 2021 17:29
@Stebalien Stebalien merged commit 88210ce into master May 10, 2021
@Stebalien Stebalien deleted the post-v1.7.1-release branch May 10, 2021 17:29
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.

2 participants