Skip to content

QD-10744: Rename products and add alternative image #291

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

Merged
merged 8 commits into from
Mar 26, 2025

Conversation

bindreams
Copy link
Member

No description provided.

Copy link
Collaborator

@smth-to-smb smth-to-smb left a comment

Choose a reason for hiding this comment

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

@bindreams Thank you very much for the section content. I have made several proposals to the text. As soon as you have a chance, could you please take a look at my suggestions?

Thank you very much.

topics/clang.md Outdated
@@ -247,7 +248,7 @@ As mentioned in [prerequisites](#Prepare+your+project), `qodana-clang` reads the
<p>Container mode is available for all linters; however, we recommend that you use native mode.</p>-->
<tabs group="software">
<tab title="GitHub Actions" group-key="github">
<note>This feature is in experimental mode, which means that its operation can be unstable.</note>
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
<note>This feature is experimental, which means it is actively in development and should not be used in a production environment..</note>

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Collaborator

@smth-to-smb smth-to-smb Mar 24, 2025

Choose a reason for hiding this comment

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

I think that this should be rewritten the following way:

This feature is experimental and being actively developed, which means that it should not be used in a production environment.

topics/clang.md Outdated
env:
QODANA_TOKEN: ${{ secrets.QODANA_TOKEN }}
</code-block>

<p>More configuration examples are available in the <a href="github.md"/> section.</p>
</tab>
<tab title="Jenkins" group-key="jenkins">
<note>This feature is in experimental mode, which means that its operation can be unstable.</note>
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
<note>This feature is experimental, which means it is actively in development and should not be used in a production environment.</note>

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that this should be rewritten the following way:

This feature is experimental and being actively developed, which means that it should not be used in a production environment.

topics/clang.md Outdated
@@ -370,12 +373,13 @@ As mentioned in [prerequisites](#Prepare+your+project), `qodana-clang` reads the
<p>More configuration examples are available in the <a href="jenkins.md"/> section.</p>
</tab>
<tab title="GitLab CI/CD" group-key="gitlab">
<note>This feature is in experimental mode, which means that its operation can be unstable.</note>
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<note>This feature is experimental, which means that is actively in development and should not be used in a production environment.</note>
<note>This feature is experimental, which means it is actively in development and should not be used in a production environment.</note>

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that this should be rewritten the following way:

This feature is experimental and being actively developed, which means that it should not be used in a production environment.

topics/clang.md Outdated
@@ -827,7 +836,7 @@ failureConditions:

{id="clang-feature-matrix"}

The %qp% linters provide the following %product% features:
The %qdcpp% family of linters provide the following %product% features:
Copy link
Collaborator

Choose a reason for hiding this comment

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

The Qodana for C/C++ is not a family of linters, it's just a single linter.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@bindreams bindreams marked this pull request as ready for review March 24, 2025 15:50
@smth-to-smb smth-to-smb merged commit 894b226 into 2025.1 Mar 26, 2025
1 check failed
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