-
Notifications
You must be signed in to change notification settings - Fork 934
Upgrade IESI to 4.0.4 for having a bumped file version. #1598
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
Conversation
And let dot settings get updated by JetBrains.
src/NHibernate.sln.DotSettings
Outdated
@@ -1,13 +1,16 @@ | |||
<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation"> | |||
<s:String x:Key="/Default/CodeStyle/CodeFormatting/CommonFormatter/ALIGNMENT_TAB_FILL_STYLE/@EntryValue">USE_SPACES</s:String> |
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.
Why this has been removed?
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.
And let dot settings get updated by JetBrains.
And because Rider/R# migrate it to others settings at each solution opening, which is a nuisance for many contributors.
src/NHibernate.sln.DotSettings
Outdated
@@ -1,13 +1,16 @@ | |||
<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation"> | |||
<s:String x:Key="/Default/CodeStyle/CodeFormatting/CommonFormatter/ALIGNMENT_TAB_FILL_STYLE/@EntryValue">USE_SPACES</s:String> |
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.
Why this file is even in this PR?
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.
Because previous migrations have been handled the same way, letting it go at some point in some unrelated PR.
src/NHibernate/NHibernate.csproj
Outdated
<ItemGroup> | ||
<PackageReference Include="Antlr3.CodeGenerator" Version="3.5.2-beta1"> | ||
<PrivateAssets>All</PrivateAssets> | ||
</PackageReference> | ||
<PackageReference Include="Antlr3.Runtime" Version="[3.5.1, 4.0)" /> | ||
<PackageReference Include="Iesi.Collections" Version="[4.0.3, 5.0)" /> | ||
<PackageReference Include="Iesi.Collections" Version="4.0.4" /> |
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.
Shall be [4.0.4, 5.0)
src/NHibernate/NHibernate.csproj
Outdated
<PackageReference Include="Remotion.Linq" Version="[2.1.2, 3.0)" /> | ||
<PackageReference Include="Remotion.Linq.EagerFetching" Version="[2.1.0, 3.0)" /> | ||
</ItemGroup> | ||
|
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.
Why?
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.
Done by the tooling (Rider).
Fully reverted and redone manually. |
And let dot settings get updated by JetBrains.