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

Version management #115

Merged
merged 4 commits into from
Apr 16, 2020
Merged

Conversation

alexhauser
Copy link
Member

Closes #101.

Description:

This updates the GitHubHelper class to handle the agreed upon version management scheme. The client now looks at it's own assembly version number when checking for updates instead of reading the version file. I left the writing of the version file in the installer though for backwards compatibility, we can remove this once we're down a few versions with both the installer and the client.

Notes:

This also adds a more convenient way for authenticating to Github for development. Just place a file called GithubAuthToken.txt containing a Github personal access token in the format of token 5c19b5ada557335de35ed54642c363f82ac1da18 in the root directory of the installer project. It will then be copied to the output directory during build and is used by the installer if present. I also added this file to the .gitignore file, so it should never get uploaded to Github.

@alexhauser alexhauser requested a review from josegomez April 16, 2020 12:30
@alexhauser alexhauser self-assigned this Apr 16, 2020
@alexhauser alexhauser added this to the v0.1.2 milestone Apr 16, 2020
Copy link
Collaborator

@josegomez josegomez left a comment

Choose a reason for hiding this comment

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

Awesome I love the token fix too
Thank yo
Merge away!

@alexhauser alexhauser merged commit fcbdb4a into sqrldev:master Apr 16, 2020
@alexhauser alexhauser deleted the version_management branch April 25, 2020 10:10
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.

Executable version management
2 participants