Skip to content

Conversation

@kmichalikk
Copy link
Contributor

@kmichalikk kmichalikk commented Nov 26, 2025

Closes https://github.com/software-mansion/react-native-screens-labs/issues/587

Description

This PR adds missing #pragma once calls at the top of the header files for consistency.

Changes

Added #pragma once here and there.

Test code and steps to reproduce

Should be working exactly the same.

Copy link
Contributor

@t0maboro t0maboro left a comment

Choose a reason for hiding this comment

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

Note: I just checked the result with the script below, I didn't test the runtime

#!/bin/bash

SEARCH_DIR="."

find "$SEARCH_DIR" -type f \
    -name '*.h' \
    -not -path '*/node_modules/*' \
    -not -path '*/Pods/*' \
    -not -path '*/bundle/*' \
    -not -path '*/build/*' \
    -print0 | \
xargs -0 grep -L -i '#pragma once'

@kmichalikk kmichalikk self-assigned this Nov 26, 2025
Copy link
Member

@kkafar kkafar left a comment

Choose a reason for hiding this comment

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

Looks good!

@kmichalikk kmichalikk merged commit 2381457 into main Nov 27, 2025
6 checks passed
@kmichalikk kmichalikk deleted the @kmichalikk/missing-pragma-in-headers branch November 27, 2025 11:45
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.

5 participants