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.

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

Discuss package structure change #49

Open
Husterknupp opened this issue Jul 16, 2014 · 4 comments
Open

Discuss package structure change #49

Husterknupp opened this issue Jul 16, 2014 · 4 comments
Milestone

Comments

@Husterknupp
Copy link
Collaborator

Problem Description
We have only one main package cc.commandmanager.core that contains all of our 14 classes. These classes are

  • exceptions as well as
  • those who do the main work (e.g., CommandManager, CommandGraph) and some
  • kind of helper classes (e.g., Optional or ResultState).

This variety might be confusing.

@FRosner, @hinneburg do you think it would be reasonable to seperate the classes into more than just the core package? Or would it produce too much overhead? What criteria make sense to split the classes with respect to further extension?

@FRosner FRosner added this to the 2.0 milestone Jul 17, 2014
@FRosner
Copy link
Collaborator

FRosner commented Jul 17, 2014

I moved this into 2.0. The structure is fine for now as we don't offer much functionality.

What does cc stand for anyway?

@Husterknupp
Copy link
Collaborator Author

It might stand for creative common and emphasizes the open character of this project. But I'm not totally sure on that.

@FRosner
Copy link
Collaborator

FRosner commented Jul 24, 2014

Very open indeed...

@FRosner
Copy link
Collaborator

FRosner commented Jul 24, 2014

@hinneburg do you want to publish the command manager under creative commons? If not, we should change the package name, I believe.

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

No branches or pull requests

2 participants