Skip to content

Conversation

@jpastoor
Copy link
Collaborator

@jpastoor jpastoor commented May 2, 2016

Expanded a bit on the API's functionality for dealing with Project Versions.
Methods are unittested and work for the current live version of JIRA hosted.

@aik099
Copy link
Member

aik099 commented May 2, 2016

Please fix the build. I guess you're using PHP 5.5 syntax.

@jpastoor jpastoor force-pushed the feature/updateVersion branch from e6a9eef to 04ee86b Compare May 2, 2016 09:43
* @param array $params Key->Value list to update the version with.
*
* @return Result|false
*/
Copy link
Member

Choose a reason for hiding this comment

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

Please be consistent about { usage in method declaration. Other methods place { on next line.

I recommend forcing IDE to check for such stuff.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Member

Choose a reason for hiding this comment

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

In this method yes, but you've added other methods as well and they have same problem.

@jpastoor jpastoor force-pushed the feature/updateVersion branch from 04ee86b to d2e695a Compare May 2, 2016 09:44
}

$params = array_merge(
array(
Copy link
Member

Choose a reason for hiding this comment

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

Double quotes are not needed in here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@jpastoor
Copy link
Collaborator Author

jpastoor commented May 2, 2016

Fixed style issues, will do a rebase before landing.

@jpastoor jpastoor force-pushed the feature/updateVersion branch from 49c017d to 6a39975 Compare May 2, 2016 10:06
@jpastoor jpastoor merged commit 7b7adf3 into console-helpers:master May 2, 2016
@jpastoor jpastoor deleted the feature/updateVersion branch May 2, 2016 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants