Skip to content
This repository has been archived by the owner on Nov 26, 2017. It is now read-only.

Add extra docs to Github package; light refactoring; bug fixes. #295

Merged
merged 3 commits into from
Dec 5, 2013
Merged

Add extra docs to Github package; light refactoring; bug fixes. #295

merged 3 commits into from
Dec 5, 2013

Conversation

eddieajau
Copy link
Contributor

  • Rename Package and GithubObject classes to AbstractPackage and
    AbstractGithubObject respectively.
  • Deprecated Package and GithubObject classes.
  • Fix bug in create tag API.
  • Fix bug in Teams::chekRepo API (missing repo owner name - this is
    technically not backward compatible, but the old method would not
    anyway)
  • Added support for releases.
  • Added more examples to documentation.

* Rename `Package` and `GithubObject` classes to `AbstractPackage` and
`AbstractGithubObject` respectively.
* Deprecated `Package` and `GithubObject` classes.
* Fix bug in create tag API.
* Fix bug in `Teams::chekRepo` API (missing repo owner name - this is
technically not backward compatible, but the old method would not
anyway)
* Added support for releases.
* Added more examples to documentation.
$action = $this->maps[$action];
}

$method =sprintf('do%s', $action);
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a missing space = sprintf

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That file should not have been included in the PR. I've removed it.

@dongilbert
Copy link
Contributor

This is fully BC, correct? I see you kept the original classes, moved the contents to follow our naming convention to Abstract*, and then extended the original class from the newly named class. Seems BC to me.

@florianv
Copy link
Contributor

florianv commented Dec 4, 2013

@dongilbert Is there any plan to open the 2.0 branch already ?

@dongilbert
Copy link
Contributor

I think so. We've been discussing possible changes for 2.0 since beta2. :) I don't see any reason why not. I'll open it to the team for discussion. 


Sent from Mailbox for iPhone

On Wed, Dec 4, 2013 at 4:27 PM, Florian Voutzinos
[email protected] wrote:

@dongilbert Is there any plan to open the 2.0 branch already ?

Reply to this email directly or view it on GitHub:
#295 (comment)

@florianv
Copy link
Contributor

florianv commented Dec 4, 2013

👍

@dbhurley
Copy link
Contributor

dbhurley commented Dec 4, 2013

Go for it!
On Dec 4, 2013 5:35 PM, "Florian Voutzinos" [email protected]
wrote:

[image: 👍]


Reply to this email directly or view it on GitHubhttps://github.com//pull/295#issuecomment-29853095
.

@eddieajau
Copy link
Contributor Author

Yes, I've made it fully b/c.

dongilbert added a commit that referenced this pull request Dec 5, 2013
Add extra docs to Github package; light refactoring; bug fixes.
@dongilbert dongilbert merged commit 9a85b7e into joomla:staging Dec 5, 2013
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.

4 participants