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

Fix samples/mvn/README.md after #185 #230

Merged
merged 1 commit into from
Oct 17, 2018

Conversation

szymonm
Copy link
Contributor

@szymonm szymonm commented Oct 15, 2018

After introducing #185 mainframer is a system wide command -- updating readme of mvn sample.

After introducing buildfoundation#185 mainframer is a system wide command -- updating readme of mvn sample.
@szymonm
Copy link
Contributor Author

szymonm commented Oct 15, 2018

Will you merge it @artem-zinnatullin ?

@artem-zinnatullin
Copy link
Contributor

Waiting for @ming13 review :)

We have same ./mainframer instead of mainframer going on in the other samples.

@ming13 I think it's good time for us to start migrating basic docs to 3.x (see #207), we can point people on 2.x tags or 2.x branch if they need docs for 2.x, most of the docs are backwards/forward compatible anyway

@artem-zinnatullin
Copy link
Contributor

Alright, gonna merge this and update the remaining examples

This particular change is not really breaking for the documentation as it just shows how to run something with Mainframer and could have been true with 2.x if user has an alias

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.

2 participants