Skip to content

Conversation

@softworkz
Copy link
Collaborator

The procedure for this was.

  • Download all API .md doc files from electronjs
  • Concatenate all into a single file
  • Remove all lines which do not container windows linux or macos
  • Remove all lines which do not start with ### (drops model object properties
  • Let AI process the resulting list and add the attributes accordingly

Copilot AI review requested due to automatic review settings November 15, 2025 07:56
Copilot finished reviewing on behalf of softworkz November 15, 2025 07:59

This comment was marked as spam.

@softworkz softworkz force-pushed the submit_platform_annotations branch from 75b9e85 to 72839eb Compare November 15, 2025 08:15
@FlorianRappl FlorianRappl added this to the 0.2.0 milestone Nov 15, 2025
Copy link
Collaborator

@FlorianRappl FlorianRappl left a comment

Choose a reason for hiding this comment

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

LGTM!

@softworkz
Copy link
Collaborator Author

This is insane - the page is hardly navigatable anymore.

image

Maybe you have permission: when you cliick on the 3-dot button, you should see "Hide" and next choose "Spam". Then it all goes away instantly....

@softworkz softworkz force-pushed the submit_platform_annotations branch 3 times, most recently from f7b2cb6 to 2334eff Compare November 15, 2025 12:50
@softworkz
Copy link
Collaborator Author

Okay, I've rebased this, but now it's failing due to that: #932 (comment)

@softworkz
Copy link
Collaborator Author

This happens due to the tests not having platform attributes yet.

When the test-update PR is merged, then this one will become green again 🤣

@softworkz softworkz force-pushed the submit_platform_annotations branch from 2334eff to 0580942 Compare November 16, 2025 01:53
@softworkz
Copy link
Collaborator Author

Rebased this one, let's see...

@FlorianRappl FlorianRappl merged commit f53d071 into ElectronNET:develop Nov 16, 2025
6 checks passed
@FlorianRappl FlorianRappl mentioned this pull request Nov 16, 2025
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