Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Refactor to JenkinsTray #50

Closed
wants to merge 5 commits into from
Closed

Refactor to JenkinsTray #50

wants to merge 5 commits into from

Conversation

jeff2001
Copy link
Collaborator

Hi, I have done a refactor from Hudson to Jenkins.
Let me know if this is pull request is what you were thinking and works for you.
Thanks

@zionyx
Copy link
Owner

zionyx commented Sep 14, 2015

Thanks alot. I was thinking about refactoring it to just JenkinsTray so it
matches the project name. Removing the word Tracker saves some spaces too.

BR
Lee

On Mon, Sep 14, 2015 at 22:34 jeff2001 [email protected] wrote:

Hi, I have done a refactor from Hudson to Jenkins.
Let me know if this is pull request is what you were thinking and works
for you.

Thanks

You can view, comment on, or merge this pull request online at:

#50
Commit Summary

  • Begin rename to Jenkins from Hudson
  • Missed some renames
  • Fix rename of about box back to original project name

File Changes

Patch Links:

—
Reply to this email directly or view it on GitHub
#50.

@jeff2001
Copy link
Collaborator Author

Updated to remove Tracker from the name

zionyx pushed a commit that referenced this pull request Sep 15, 2015
@jeff2001 @zionyx
#50. Begin rename to Jenkins from Hudson
90ce5da 
Missed some renames
Fix rename of about box back to original project name
Remove Tracker from name
A few more tracker renames
zionyx added a commit that referenced this pull request Sep 15, 2015
@zionyx
#50. Final touch up for JenkinsTray.
532c40f
@zionyx zionyx changed the title Jenkins rename Refactor to JenkinsTray Sep 15, 2015
@zionyx zionyx added this to the v1.0.4 milestone Sep 15, 2015
@zionyx
Copy link
Owner

zionyx commented Sep 15, 2015

Thanks alot @jeff2001. I squashed all your commit into 1 and added a minor touch up before merging to master. Great work.

Are you ready for the next assignment? 😄

@jeff2001
Copy link
Collaborator Author

Pleasure, fire away

@zionyx
Copy link
Owner

zionyx commented Sep 15, 2015

@jeff2001, please see #43

@zionyx zionyx closed this Sep 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
v1.0.4
Development

Successfully merging this pull request may close these issues.
2 participants
@jeff2001 @zionyx