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

Update run profiles #160

Merged
merged 2 commits into from
Aug 21, 2024
Merged

Update run profiles #160

merged 2 commits into from
Aug 21, 2024

Conversation

iosdev747
Copy link
Collaborator

PR changes:

  • add dependency stack run and app run profiles
  • ignore any run profile changes
  • update readme with new profiles

PS: I also tried adding references to the run profiles in readme so IntelliJ can detect them and can be run directly from readme (https://www.jetbrains.com/help/idea/markdown.html#run-commands). Making user friendly first time setup. But detection is not working for profiles.

Copy link

github-actions bot commented Aug 6, 2024

Test Results

275 tests   275 ✅  56s ⏱️
 55 suites    0 💤
 55 files      0 ❌

Results for commit c8f9fa1.

♻️ This comment has been updated with latest results.

<option value="--args=&quot;src/main/resources/configuration.yml&quot;" />
</list>
</option>
<option name="vmOptions" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

-Dvertx.disableDnsResolver=true -Dlog4j2.configurationFile=src/main/resources/log4j2.xml

former is needed on Mac and later for enabling traces.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added both, and btw for src/main/resources/log4j2.xml its being auto picked.

.run/varadhi local [run].run.xml Show resolved Hide resolved
Copy link
Collaborator

@kmrdhruv kmrdhruv left a comment

Choose a reason for hiding this comment

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

LGTM

@iosdev747 iosdev747 merged commit 848e07d into master Aug 21, 2024
3 checks passed
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