Skip to content

Feature/630 create assembly info #760

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

Merged

Conversation

Philo
Copy link
Contributor

@Philo Philo commented Jan 27, 2016

Re #630

Philo and others added 5 commits September 3, 2015 18:26
… generated

Adds argument /ensureassemblyinfo, when specified and
/updateassemblyinfofilename is specified and does not exist we will
generate a template of the assembly info file containing only *Version
attributes in either cs, fs, vb allowing the version values to be replaced
… generated

Adds argument /ensureassemblyinfo, when specified and
/updateassemblyinfofilename is specified and does not exist we will
generate a template of the assembly info file containing only *Version
attributes in either cs, fs, vb allowing the version values to be replaced
… listed

When specifying mulitple files the -ensureassemblyinfo switch can not
be specified to protect against GitVersion generating multiple files
with the same assembly attributes and breaking a build.

-updateassemblyinfo Info.cs Info1.cs
This will update any known version attributes in both files, if the file
does not exist then it will be ignored

-updateassemblyinfo Info.cs Info1.cs -ensureassemblyinfo
This will cause a warning exception to be thrown as -ensureassemblyinfo
can not be specified with multiple files

-updateassemblyinfo ..\Common\Info.cs -ensureassemblyinfo
Assuming ..\Common\Info.cs does not exist, this will generate an assembly
info file at this location based on the embedded template for the file
type (cs,vb,fs)
…itVersion into feature/630-create-assembly-info
@gep13
Copy link
Member

gep13 commented Jan 28, 2016

@Philo looks like there are some issues with the CI Build as a result of the changes that you have made. Are you in a position to look into what is going on?

Thanks!

@Philo
Copy link
Contributor Author

Philo commented Jan 28, 2016

@gep13, I'll take a look, however may not be today

@gep13
Copy link
Member

gep13 commented Jan 28, 2016

@Philo not a problem at all 👍

… have been specified on the command line
@Philo
Copy link
Contributor Author

Philo commented Feb 5, 2016

@gep13 - fixed the build now

@JakeGinnivan JakeGinnivan merged commit 4a7cf39 into GitTools:master Apr 4, 2016
@gep13
Copy link
Member

gep13 commented Apr 4, 2016

@Philo thanks for your contribution here! 👍

@JakeGinnivan
Copy link
Contributor

I think this has a bug in it.

It should only add the assembly info attributes in if the filename is specified. I think this is the cause of the assembly info issues since the last release

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.

3 participants