Skip to content

docs: Fix dashboards docs update workflow#252

Merged
izmalk merged 3 commits into
2/edgefrom
submodule-workflow-fix-upstream
Apr 24, 2026
Merged

docs: Fix dashboards docs update workflow#252
izmalk merged 3 commits into
2/edgefrom
submodule-workflow-fix-upstream

Conversation

@izmalk
Copy link
Copy Markdown
Contributor

@izmalk izmalk commented Apr 24, 2026

Fixed the docs/docs directory path failure. We need to go from docs/dashboards upwards two levels, instead of just one. Replaced with cd ../...
Fixed a potential issue with the CODEOWNERS file being empty. We check for its existence and only add the reviewers if the file exists.

Workflow Logs | Demo PR

izmalk added 2 commits April 24, 2026 12:37
Fixed docs/docs directory path failure. We need to go from docs/dashboards upwards two levels, so replaced with `cd ../..`.
Fixed a potential issue with CODEOWNERS file being empty. We check for its existence and only add the option if the file exists.
Caused by shallow cloning
@izmalk izmalk self-assigned this Apr 24, 2026
Mehdi-Bendriss
Mehdi-Bendriss previously approved these changes Apr 24, 2026
akram09
akram09 previously approved these changes Apr 24, 2026
@izmalk
Copy link
Copy Markdown
Contributor Author

izmalk commented Apr 24, 2026

Fixed a linter error.
Retested after the fix - canonical/opensearch-operator#817

@izmalk izmalk merged commit 4fad02f into 2/edge Apr 24, 2026
29 checks passed
@izmalk izmalk deleted the submodule-workflow-fix-upstream branch April 24, 2026 13:27
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.

3 participants