Skip to content

Conversation

@oleksandr-horobets
Copy link

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

Bad approach. There may be other implementations of abstract project, which are being ignored here

Copy link
Author

Choose a reason for hiding this comment

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

I don't like this approach too, but how can I obtain project builders using only AbstractProject?

Copy link
Member

Choose a reason for hiding this comment

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

Agreed. I would recommend to add a logging on the INFO level in such case. This approach may become obsolete when Template projects start referencing their users somehow.

@jenkinsadmin
Copy link
Member

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@Brantone
Copy link
Contributor

Brantone commented Feb 6, 2016

@oleksandr-horobets : Will some of these recommendations be implemented?

@Brantone
Copy link
Contributor

Still un-implemented and now with conflicts to resolve :(
The tests are nice, but lots of recommendations that should be corrected

@oleksandr-horobets
Copy link
Author

@Brantone I'm working on comments now, hope will be done soon:)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants