Skip to content

WF Create issues for muted tests: fix filepath for created issues #16631

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 1, 2025
Merged

WF Create issues for muted tests: fix filepath for created issues #16631

merged 3 commits into from
Apr 1, 2025

Conversation

naspirato
Copy link
Collaborator

@naspirato naspirato commented Apr 1, 2025 •
edited
Loading

Changelog entry

Fixed filepath for created issues and usage of var in workflow

Summary
The changes improve the reliability and robustness of the script by adding necessary environment checks and using output variables.
The workflow updates ensure that the dynamically generated file paths are correctly referenced in subsequent steps.
Overall, these changes enhance the maintainability and correctness of the workflow.

Changelog category

  • Not for changelog (changelog entry is not required)

Description for reviewers

...

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

🟢 2025-04-01 17:35:14 UTC The validation of the Pull Request description is successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

⚪ 2025-04-01 17:28:39 UTC Pre-commit check linux-x86_64-relwithdebinfo for 0f87fd7 has started.
⚪ 2025-04-01 17:28:55 UTC Artifacts will be uploaded here
⚪ 2025-04-01 17:31:12 UTC ya make is running...
🟢 2025-04-01 17:31:17 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-04-01 17:31:23 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

⚪ 2025-04-01 17:29:29 UTC Pre-commit check linux-x86_64-release-asan for 0f87fd7 has started.
⚪ 2025-04-01 17:29:46 UTC Artifacts will be uploaded here
⚫ 2025-04-01 17:31:47 UTC Check cancelled

@naspirato naspirato requested a review from maximyurchuk April 1, 2025 17:34
@naspirato naspirato requested a review from iddqdex April 1, 2025 17:34
@naspirato naspirato marked this pull request as ready for review April 1, 2025 17:34
@naspirato naspirato enabled auto-merge (squash) April 1, 2025 17:34
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

⚪ 2025-04-01 17:35:53 UTC Pre-commit check linux-x86_64-release-asan for 34164f5 has started.
⚪ 2025-04-01 17:36:08 UTC Artifacts will be uploaded here
⚪ 2025-04-01 17:38:34 UTC ya make is running...
🟢 2025-04-01 17:38:39 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-04-01 17:38:47 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 1, 2025 •
edited
Loading

⚪ 2025-04-01 17:35:58 UTC Pre-commit check linux-x86_64-relwithdebinfo for 34164f5 has started.
⚪ 2025-04-01 17:36:13 UTC Artifacts will be uploaded here
⚪ 2025-04-01 17:38:34 UTC ya make is running...
🟢 2025-04-01 17:38:39 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-04-01 17:38:47 UTC Build successful.

@naspirato naspirato merged commit cc8686f into main Apr 1, 2025
16 checks passed
@naspirato naspirato mentioned this pull request Apr 1, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iddqdex iddqdex iddqdex approved these changes

@maximyurchuk maximyurchuk Awaiting requested review from maximyurchuk

Assignees
No one assigned
Labels
not-for-changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@naspirato @iddqdex