Skip to content

chore: Rename folders to match crate names#733

Open
PoignardAzur wants to merge 1 commit into
AccessKit:mainfrom
PoignardAzur:rename_folders
Open

chore: Rename folders to match crate names#733
PoignardAzur wants to merge 1 commit into
AccessKit:mainfrom
PoignardAzur:rename_folders

Conversation

@PoignardAzur

Copy link
Copy Markdown
Contributor

No description provided.

@DataTriny DataTriny changed the title Rename folders to match crate names chore: Rename folders to match crate names Jun 27, 2026
@PoignardAzur

Copy link
Copy Markdown
Contributor Author

@DataTriny Any chance you could give this a quick look? It's a 20-lines PR, renames excluded.

@DataTriny DataTriny left a comment

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.

If the goal is to improve discoverability in the project, what do you think about renaming the "platforms" directory to "adapters"?

@@ -1 +1 @@
{"common":"0.24.1","consumer":"0.37.0","platforms/macos":"0.26.2","platforms/windows":"0.33.1","platforms/winit":"0.33.1","platforms/unix":"0.22.0","platforms/atspi-common":"0.19.0","platforms/android":"0.7.4","platforms/ios":"0.1.1"} No newline at end of file
{"accesskit":"0.24.1","consumer":"0.37.0","platforms/macos":"0.26.2","platforms/windows":"0.33.1","platforms/winit":"0.33.1","platforms/unix":"0.22.0","platforms/atspi-common":"0.19.0","platforms/android":"0.7.4","platforms/ios":"0.1.1"} No newline at end of file

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.

In this file accesskit_consumer is only referred to as "consumer". This is going to mess up release-please.

@PoignardAzur

Copy link
Copy Markdown
Contributor Author

what do you think about renaming the "platforms" directory to "adapters"?

Probably a good idea, given the discussions we had in the ARCHITECTURE.md PR.

In this file accesskit_consumer is only referred to as "consumer". This is going to mess up release-please.

I'll take a look.

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