Skip to content

Project configuration cleanup#3

Merged
arakis merged 2 commits intoSharpKit:masterfrom
hultqvist:master
Jul 16, 2014
Merged

Project configuration cleanup#3
arakis merged 2 commits intoSharpKit:masterfrom
hultqvist:master

Conversation

@hultqvist
Copy link
Copy Markdown

This is the result if #2

All project files in Defs now only have one configuration: Debug/AnyCPU

Fixed or ignored various warnings mostly related to malformed comments.

@arakis
Copy link
Copy Markdown
Member

arakis commented Jul 16, 2014

Commit 14d332e seems to be fine. Commit bf81251 is mostly ok, too, but maye we shuld discuss about it first.

You're right, the chars <, > and so on have to be written down in encoded form. But fact is, that this was "never" a problem for visual studio or monodevelop (as i remind). Both IDEs shows the chars like expected. Only a warning will be shown. On the other side, avoiding the html encoding makes the comments more readable / modifyable in the source code.

The specific warning ("Error code") can be supressed.

Your other comment fixes seems to be ok (removed unneeded spaces, other attribute names and so on).

@arakis
Copy link
Copy Markdown
Member

arakis commented Jul 16, 2014

I just checked the comments behavior. They need to be encoded. So, the merge is fine.

arakis added a commit that referenced this pull request Jul 16, 2014
Project configuration cleanup
@arakis arakis merged commit 602e113 into SharpKit:master Jul 16, 2014
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