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

Mavenization #1

Open
RodionGork opened this issue Feb 14, 2014 · 3 comments
Open

Mavenization #1

RodionGork opened this issue Feb 14, 2014 · 3 comments

Comments

@RodionGork
Copy link
Owner

The project was created when I know not of Maven. This should be fixed :)

@trirano
Copy link

trirano commented Jan 3, 2016

Hi Rodion,

I have mavenized the project but I am out of depth with github.

I have just created a pull request.
If you could give me some feedback, I would appreciate.

Have a nice day

@cosmocracy
Copy link

I would be happy to independently verify that the PR for this works. Just let me know if you intend to consider merging it, @RodionGork, and I'll give it a try.

@cosmocracy
Copy link

PS, @trirano, you did fine. I'd note/suggest that the content you added to your Mavenization README would better live inside the PR itself since it's transient (i.e., won't matter after the PR is merged) and is also more visible to those considering the PR that way. Regardless, good work and thank you for heping this project!

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

3 participants