Skip to content
This repository was archived by the owner on Oct 31, 2021. It is now read-only.

Conversation

vasily-kirichenko
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhap rename this to ProjectDependencies and add [<RequireQualifiedAccess>] (all case names here are really common).

@vasily-kirichenko
Copy link
Contributor Author

Done.

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps extract this https://github.com/fsprojects/VisualFSharpPowerTools/blob/master/src/FSharpVSPowerTools.Logic/NavigateToItem.fs#L149-L162 as a utility function and use it here.

It handles projects in solution items as our test projects in test folders.

@dungpa
Copy link
Contributor

dungpa commented Apr 7, 2014

Great jobs!

Just one more comment above.

fixed: Rename refactoring does not work for fsx files
@vasily-kirichenko
Copy link
Contributor Author

Done. The issue with *.fsx files is also fixed.

dungpa added a commit that referenced this pull request Apr 7, 2014
@dungpa dungpa merged commit 22cba45 into fsprojects-archive:master Apr 7, 2014
@7sharp9
Copy link

7sharp9 commented Apr 7, 2014

I wish you guys would send some refactoring love over to Xamarin studio!

Good work

@vasily-kirichenko vasily-kirichenko deleted the solution-wide-renaming branch April 8, 2014 04:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants