-
Notifications
You must be signed in to change notification settings - Fork 5
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
Adjust/remove obsolete links #2
Conversation
084a979
to
8d5b3e0
Compare
@marchof could you please review? |
@@ -55,11 +53,7 @@ <h1 style="clear:left">Resources</h1> | |||
<h2>Articles About EclEmma</h2> | |||
|
|||
<ul> | |||
<li>Tests für die Tests by Marc R. Hoffmann, <a class="extern" href="http://it-republik.de/jaxenter/eclipse-magazin-ausgaben/Android-000442.html">Eclipse Magazin Vol. 3.2011</a>, 2011/03/25 [de]</li> |
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.
As these paper publications still exist I wouldn't remove them from the list. Just the web links should be 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.
@marchof makes sense, fixed
<li><a class="extern" href="http://agile.csc.ncsu.edu/SEMaterials/tutorials/eclemma/">Test Coverage with EclEmma</a> by Laurie Williams, Ben Smith and Sarah Heckman, <a class="extern" href="http://www.ncsu.edu/">North Carolina State University</a>, 2009/03/30</li> | ||
<li>Testabdeckung mit Open-Source-Tools by Tim Wellhausen, <a class="extern" href="http://www.sigs-datacom.de/sd/publications/js/2008/06/index.htm">JavaSpektrum</a>, 2008/06/01 [de]</li> |
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.
Also JavaSpektum is a printed magazine, so I prefer to keep the reference.
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.
@marchof fixed
@Godin Thanks! |
I've got an email :
Response was - to use mailing list in future for such notifications.
But anyway we should do a check followed by a fix.