FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added a new option 'NrtProcessAnyProperty'. by shuebener · Pull Request #3 · ygoe/NetRevisionTask · GitHub

Added a new option 'NrtProcessAnyProperty'. - #3

Open
shuebener wants to merge 1 commit into
ygoe:masterfrom
shuebener:master
Open

Added a new option 'NrtProcessAnyProperty'.#3
shuebener wants to merge 1 commit into
ygoe:masterfrom
shuebener:master

Conversation

Copy link
Copy Markdown

Added a new option 'NrtProcessAnyProperty' to process format strings in any property.
So it's possible to add more defined custom attributes like this:

[assembly: AssemblyGitRevision("{CHASH}{!:*}")]
[assembly: AssemblyBuildTime("{b:uymd-} {b:uhms:}")]

ygoe commented May 6, 2019

Copy link
Copy Markdown
Owner

Sorry, I hadn't seen this. It's an interesting idea.

But these [Things] in brackets are called attributes, not properties, so the new option is not named correctly. Also, I'd align the name with the existing options and rather call it allAttributes and NrtAllAttributes.

BTW, if(newline != lines[i]) in AssemblyInfoHelper.cs is missing a space.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL