JMediawiki package #1547

wants to merge 11 commits into


None yet
4 participants

nprasath002 commented Sep 26, 2012

JMediawiki package with unit tests

@elinw elinw commented on an outdated diff Sep 26, 2012

@@ -0,0 +1,202 @@
+ * @package Joomla.Platform
+ * @subpackage MediaWiki
+ *
+ * @copyright Copyright (C) 2005 - 2012 Open Source Matters, Inc. All rights reserved.
+ * @license GNU General Public License version 2 or later; see LICENSE
+ */
+defined('JPATH_PLATFORM') or die;
+ * Joomla Platform class for interacting with a GitHub server instance.

elinw Sep 26, 2012


Here and a few other places you have GitHub not mediawiki


nprasath002 commented Sep 28, 2012

corrected all typos. thanks


pasamio commented Oct 1, 2012

Looks like the unit tests have errors:


LouisLandry commented Oct 9, 2012

@nprasath002 I'd love to get this ready for merging. Can you get the unit test and code style issues sorted out for us so we can give it a proper once over?

Note: I'm closing this pull request for now until the requested changes are uploaded. Please re-open the pull request at that time so we can review it again. Thanks.

LouisLandry closed this Oct 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment