Skip to content

Conversation

@kwin
Copy link
Member

@kwin kwin commented Mar 11, 2022

No description provided.

Copy link
Member

@slawekjaranowski slawekjaranowski left a comment

Choose a reason for hiding this comment

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

  • do we need some of special dependency for project in order to use embedded mode?
  • example of setting mave.home propery by failsafe will be useful, like
<plugin>
    <artifactId>maven-failsafe-plugin</artifactId>
    <configuration>
        <systemPropertyVariables>
            <maven.home>${maven.home}</maven.home>
        </systemPropertyVariables>
    </configuration>
</plugin>

@kwin kwin force-pushed the feature/MSHARED-871-initial-documentation branch from f71cf5c to 6ae0fdb Compare March 11, 2022 20:16
@kwin kwin requested a review from slawekjaranowski March 11, 2022 20:17
@slawekjaranowski
Copy link
Member

pleas also add menu item under OVERVIEW section.

@kwin kwin force-pushed the feature/MSHARED-871-initial-documentation branch from 6ae0fdb to fe83a42 Compare March 12, 2022 14:49
@kwin kwin force-pushed the feature/MSHARED-871-initial-documentation branch 2 times, most recently from 2da6bdf to 9925458 Compare March 13, 2022 10:30
Copy link
Member

@hboutemy hboutemy left a comment

Choose a reason for hiding this comment

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

nice work done, thank you
can be merged now or with the few little enhancements proposed

@slawekjaranowski slawekjaranowski self-assigned this Mar 13, 2022
@kwin kwin force-pushed the feature/MSHARED-871-initial-documentation branch from 9925458 to 1d1d15c Compare March 13, 2022 17:00
@slawekjaranowski slawekjaranowski merged commit 07e7423 into apache:master Mar 13, 2022
@slawekjaranowski slawekjaranowski added the documentation Improvements or additions to documentation label Mar 15, 2022
@jira-importer
Copy link

Resolve #145

1 similar comment
@jira-importer
Copy link

Resolve #145

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants