Skip to content

Add short_file_name placeholder to logging configuration options#6226

Merged
fujitatomoya merged 3 commits intoros2:rollingfrom
botsandus:add-short-file-name
Feb 21, 2026
Merged

Add short_file_name placeholder to logging configuration options#6226
fujitatomoya merged 3 commits intoros2:rollingfrom
botsandus:add-short-file-name

Conversation

@tonynajjar
Copy link
Contributor

@tonynajjar tonynajjar commented Feb 11, 2026 •
edited
Loading

Description

Docs for ros2/rcutils#541

Did you use Generative AI?

Additional Information

@tonynajjar
Add short_file_name placeholder to logging configuration options
805b721 
Signed-off-by: Tony Najjar <tony.najjar@dexory.com>
@tonynajjar tonynajjar mentioned this pull request Feb 11, 2026
Merged
@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 13, 2026

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Feb 13, 2026

rebase

❌ Pull request can't be updated with latest base branch changes

Details

Mergify needs the author permission to update the base branch of the pull request.
@tonynajjar needs to authorize modification on its head branch.

@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 13, 2026

Important

please hold this PR until ros2/rcutils#541 is merged.

@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 13, 2026

@tonynajjar can you rebase? i tried but i dont seem to have the permission to the dev repo.

@tonynajjar
Copy link
Contributor Author

tonynajjar commented Feb 19, 2026

@tonynajjar can you rebase? i tried but i dont seem to have the permission to the dev repo.

Done

@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 21, 2026

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Feb 21, 2026

rebase

❌ Pull request can't be updated with latest base branch changes

Details

Mergify needs the author permission to update the base branch of the pull request.
@tonynajjar needs to authorize modification on its head branch.

@fujitatomoya fujitatomoya enabled auto-merge (squash) February 21, 2026 02:51
@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 21, 2026

@tonynajjar i do not have permission to rebase this, i need your help to do that... i did enable auto-merge, so once this is rebased, it will run the workflow and if that goes green, this PR will be merged automatically.

@tonynajjar
Copy link
Contributor Author

tonynajjar commented Feb 21, 2026

@fujitatomoya done

@fujitatomoya fujitatomoya merged commit a6b0ba7 into ros2:rolling Feb 21, 2026
6 checks passed
@mergify mergify bot mentioned this pull request Feb 22, 2026
Merged
@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Feb 22, 2026

@Mergifyio backport kilted

@mergify
Copy link
Contributor

mergify bot commented Feb 22, 2026 •
edited
Loading

backport kilted

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Feb 22, 2026
@tonynajjar @mergify
Add short_file_name placeholder to logging configuration options (#6226)
97b6ccd 
Signed-off-by: Tony Najjar <tony.najjar@dexory.com>
(cherry picked from commit a6b0ba7)
@mergify mergify bot mentioned this pull request Feb 22, 2026
Merged
ahcorde pushed a commit that referenced this pull request Feb 23, 2026
@mergify @tonynajjar
Add short_file_name placeholder to logging configuration options (#6226…
fa1edaa 
…) (#6254)

(cherry picked from commit a6b0ba7)

Signed-off-by: Tony Najjar <tony.najjar@dexory.com>
Co-authored-by: Tony Najjar <tony.najjar@dexory.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kscottz kscottz kscottz approved these changes

@fujitatomoya fujitatomoya fujitatomoya approved these changes

Assignees

@tonynajjar tonynajjar

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tonynajjar @fujitatomoya @kscottz