-
Notifications
You must be signed in to change notification settings - Fork 102
Koog Docs with Knit #542
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
Koog Docs with Knit #542
Conversation
Qodana for JVM387 new problems were found
@@ Code coverage @@
+ 68% total lines covered
8831 lines analyzed, 6024 lines covered
# Calculated according to the filters of your coverage tool ☁️ View the detailed Qodana report Contact Qodana teamContact us at [email protected]
|
0bb7bee
to
ca65312
Compare
3f7124f
to
50f72c3
Compare
f459b29
to
6126682
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I would use includeBuild("docs")
to exclude the docs from the main build
2ef1492
to
2e5391e
Compare
This PR brings the following changes:
! The docs/docs changes are already reviewed by tech writers, please take a look into knit integration only and execution (CI) only
! The CI is tested on current branch, check https://github.com/JetBrains/koog/actions/runs/16755658514/job/47437479580
! I'm ready to change docs publication from this repo, but I can not do it right now as gh-pages have outdated docs from main branch
Type of the change
Checklist for all pull requests
develop
as the base branchAdditional steps for pull requests adding a new feature