Skip to content
This repository was archived by the owner on Dec 2, 2021. It is now read-only.

Use file version#22

Closed
asherber wants to merge 8 commits into304NotModified:masterfrom
asherber:use-file-version
Closed

Use file version#22
asherber wants to merge 8 commits into304NotModified:masterfrom
asherber:use-file-version

Conversation

@asherber
Copy link

This branch addresses #13 by adding two config options:

  1. OverwriteFileVersion is set to true by default. If it is set to false, Stamp will not overwrite the AssemblyFileVersion with the AssemblyVersion.
  2. UseFileVersion is false by default. If it is set to true, Stamp will use AssemblyFileVersion to construct the AssemblyInformationalVersion, rather than using AssemblyVersion. Also, if this is set to true, OverwriteFileVersion will be false and will ignore any explicitly set value.

I included tests for both of these options that mimic TaskTests.

@SimonCropp
Copy link
Collaborator

this project is no longer maintained. raise a new issue if u want to take ownership

@304NotModified
Copy link
Owner

304NotModified commented Aug 30, 2018

Is this backwards compatible?

(PS I'm the owner since today)

@asherber
Copy link
Author

I believe the changes I made were backwards compatible. But frankly, it was two years ago and I haven't used this tool in a long time, so I wouldn't swear to anything.

@304NotModified 304NotModified added this to the 1.5 milestone Aug 30, 2018
- `%branch%` is replaced with the branch name of the repository
- `%haschanges%` is replaced with the string defined in the ChangeString attribute in the configuration, see below.
- `%user%` is replaced with the current user name
- `%machinename%` is replaced with the current machine name
Copy link
Owner

Choose a reason for hiding this comment

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

Already added these to the readme. Thanks!

@304NotModified
Copy link
Owner

FYI, I like to merge this one soon

(need to fix curl first on Windows first, to get this merged locally)

@304NotModified 304NotModified removed this from the 1.5 milestone Sep 3, 2018
@304NotModified
Copy link
Owner

304NotModified commented Sep 3, 2018

superseded by #48 (squashed/rebased version). Thanks @asherber

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments