Skip to content

Conversation

dmyger
Copy link
Collaborator

@dmyger dmyger commented Jul 10, 2025

Turn on static code analyzes.

Second part for #1184

Part of #TNTP-3105

@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from 2b4a206 to 59e26ec Compare July 10, 2025 13:03
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 58058fc to 45cce06 Compare July 10, 2025 13:04
@dmyger dmyger added the full-ci Enables full ci tests label Jul 10, 2025
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from 59e26ec to e155128 Compare July 11, 2025 12:45
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 45cce06 to 9a85455 Compare July 11, 2025 12:46
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from e155128 to b4547b7 Compare July 11, 2025 13:19
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 9a85455 to 0beb8b5 Compare July 11, 2025 13:20
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from b4547b7 to 20a96d7 Compare July 16, 2025 10:27
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 0beb8b5 to ca78a15 Compare July 16, 2025 10:27
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from 20a96d7 to 707afff Compare July 22, 2025 10:17
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from ca78a15 to 8657ea5 Compare July 22, 2025 10:18
Copy link
Contributor

@elhimov elhimov left a comment

Choose a reason for hiding this comment

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

Sorry, but I don't catch the point of splitting into 2 PRs. The first one (#1184) turns off the job and the second one (this PR) turns it back. I would understand if this job fails in #1184 and then I would expect that it is fixed with this PR and turned back, but I see no fix, it just turned back. Would you please clarify? Why don't we just keep this job intact in #1184 and drop this PR?

@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch 2 times, most recently from 25a4d6e to 89506dd Compare July 23, 2025 17:35
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 8657ea5 to c1fa1f1 Compare July 23, 2025 17:36
@dmyger
Copy link
Collaborator Author

dmyger commented Jul 23, 2025

Sorry, but I don't catch the point of splitting into 2 PRs.

Please check comment: #1184 (comment)

@elhimov
Copy link
Contributor

elhimov commented Jul 24, 2025

Sorry, but I don't catch the point of splitting into 2 PRs.

Please check comment: #1184 (comment)

Got it, thanks!

@elhimov elhimov self-requested a review July 24, 2025 13:30
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from 89506dd to da4751e Compare July 24, 2025 14:12
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from c1fa1f1 to 40b47f9 Compare July 24, 2025 14:13
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from da4751e to 0e01c6d Compare July 24, 2025 14:22
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 40b47f9 to 8ff567b Compare July 24, 2025 14:23
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 8ff567b to 55ecb7c Compare July 25, 2025 10:18
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code branch from 0e01c6d to 400e4db Compare July 25, 2025 11:15
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from 55ecb7c to caddb4b Compare July 25, 2025 11:16
Base automatically changed from dmyger/tntp-3105_turn_on_spell_checker_for_code to master July 25, 2025 15:25
Part of #TNTP-3105
@dmyger dmyger force-pushed the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch from caddb4b to d2cdc88 Compare July 25, 2025 15:27
@dmyger dmyger merged commit 63c4c51 into master Jul 25, 2025
33 of 34 checks passed
@dmyger dmyger deleted the dmyger/tntp-3105_turn_on_spell_checker_for_code_p2 branch July 25, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables full ci tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants