Skip to content

refactor(fspy): change AccessMode to bitflags#36

Merged
branchseer merged 2 commits intomainfrom
11-19-refactor_fspy_change_accessmode_to_bitflags
Nov 19, 2025
Merged

refactor(fspy): change AccessMode to bitflags#36
branchseer merged 2 commits intomainfrom
11-19-refactor_fspy_change_accessmode_to_bitflags

Conversation

@branchseer
Copy link
Copy Markdown
Member

No description provided.

@branchseer branchseer marked this pull request as ready for review November 18, 2025 16:36
Copy link
Copy Markdown
Member Author

branchseer commented Nov 18, 2025

bitflags! {
// Attributes can be applied to flags types
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
pub struct Flags: u32 {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this left by accident?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, thanks for noticing. Let me remove it.

@branchseer branchseer changed the base branch from 11-18-fix_fspy_correctly_report_access_mode_from_node_fs_on_windows to graphite-base/36 November 19, 2025 01:25
@branchseer branchseer force-pushed the 11-19-refactor_fspy_change_accessmode_to_bitflags branch from dd2edf0 to bc58496 Compare November 19, 2025 01:25
@graphite-app graphite-app bot changed the base branch from graphite-base/36 to main November 19, 2025 01:26
@branchseer branchseer force-pushed the 11-19-refactor_fspy_change_accessmode_to_bitflags branch from bc58496 to bd1e224 Compare November 19, 2025 01:26
@branchseer branchseer force-pushed the 11-19-refactor_fspy_change_accessmode_to_bitflags branch from bd1e224 to 87b225f Compare November 19, 2025 01:32
@branchseer branchseer merged commit ede2325 into main Nov 19, 2025
6 checks passed
@Brooooooklyn Brooooooklyn deleted the 11-19-refactor_fspy_change_accessmode_to_bitflags branch December 11, 2025 06:39
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.

2 participants