-
Notifications
You must be signed in to change notification settings - Fork 616
Conversation
Will merge this after Release 43. |
Conflicts: appshell/version.rc installer/mac/buildInstaller.sh installer/win/brackets-win-install-build.xml
Ready to rock! Thanks @marcelgerber. |
@@ -43,7 +43,7 @@ BEGIN | |||
BEGIN | |||
VALUE "CompanyName", "brackets.io\0" | |||
VALUE "FileDescription", "\0" | |||
VALUE "FileVersion", "Sprint 44\0" | |||
VALUE "FileVersion", "Release 44\0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn. It should be Release 0.44, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, how could I have missed this... ;-) Let's change it and submit the change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you like to fix it or should I do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can fix and submit it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcelgerber I have fixed it with #465
For https://trello.com/c/atMTdx6l/1313-brackets-7828-should-be-consistent-about-release-vs-sprint-issue.
Similar to adobe/brackets#8680.
@redmunds