Skip to content

Conversation

@KiLLeRRaT
Copy link
Contributor

@KiLLeRRaT KiLLeRRaT commented Mar 1, 2020

  • Added tests that fail if we don't have a newline after Assembly attributes
  • Updated FileHelper to support commented out Assembly attributes that do not end with a new line

Fixes #1939 and #1802.

- Added tests that fail if we don't have a newline after Assembly attributes
- Updated FileHelper to support commented out Assembly attributes that do not end with a new line
Copy link
Member

@asbjornu asbjornu left a comment

Choose a reason for hiding this comment

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

LGTM!

@arturcic
Copy link
Member

arturcic commented Mar 2, 2020

Will this fix this issue as well? #1802

@asbjornu
Copy link
Member

asbjornu commented Mar 2, 2020

I think we should merge this and see whether #2102 turns green. 👍

@arturcic
Copy link
Member

arturcic commented Mar 2, 2020

Ok @asbjornu

@asbjornu asbjornu merged commit 955ff5e into GitTools:master Mar 2, 2020
@asbjornu
Copy link
Member

asbjornu commented Mar 2, 2020

Thank you for your contributions @KiLLeRRaT! 🙏

@KiLLeRRaT KiLLeRRaT deleted the EOFRegex branch March 2, 2020 18:18
@KiLLeRRaT
Copy link
Contributor Author

@asbjornu Not a problem, happy to help.

Eagerly waiting on some of the other issues/PRs to be finalised and sorted out so I can upgrade my projects to ust 5.1.4. (I'm stuck with .NET472, build issues etc. Should be dealt to with some of the other PRs).

Have a good one!

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.

Commented out AssemblyVersion tags still causing GitVersion to not generate versions correctly

3 participants